fides icon indicating copy to clipboard operation
fides copied to clipboard

[Manual Task PR #3] Email for external user invitation

Open galvana opened this issue 5 months ago • 2 comments

Closes []

Description Of Changes

Write some things here about the changes and any potential caveats

Code Changes

  • list your code changes here

Steps to Confirm

  1. list any manual steps for reviewers to confirm the changes

Pre-Merge Checklist

  • [ ] Issue requirements met
  • [ ] All CI pipelines succeeded
  • [ ] CHANGELOG.md updated
    • [ ] Add a https://github.com/ethyca/fides/labels/db-migration label to the entry if your change includes a DB migration
    • [ ] Add a https://github.com/ethyca/fides/labels/high-risk label to the entry if your change includes a high-risk change (i.e. potential for performance impact or unexpected regression) that should be flagged
    • [ ] Updates unreleased work already in Changelog, no new entry necessary
  • Followup issues:
    • [ ] Followup issues created
    • [ ] No followup issues
  • Database migrations:
    • [ ] Ensure that your downrev is up to date with the latest revision on main
    • [ ] Ensure that your downgrade() migration is correct and works
      • [ ] If a downgrade migration is not possible for this change, please call this out in the PR description!
    • [ ] No migrations
  • Documentation:
    • [ ] Documentation complete, PR opened in fidesdocs
    • [ ] Documentation issue created in fidesdocs
    • [ ] If there are any new client scopes created as part of the pull request, remember to update public-facing documentation that references our scope registry
    • [ ] No documentation updates required

galvana avatar Jun 25 '25 00:06 galvana

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
fides-plus-nightly ⬜️ Ignored (Inspect) Visit Preview Jul 2, 2025 4:00pm
fides-privacy-center ⬜️ Ignored (Inspect) Jul 2, 2025 4:00pm

vercel[bot] avatar Jun 25 '25 00:06 vercel[bot]

Codecov Report

Attention: Patch coverage is 58.33333% with 10 lines in your changes missing coverage. Please review.

Project coverage is 86.77%. Comparing base (468a2b2) to head (ca2d0cf).

Files with missing lines Patch % Lines
.../api/service/messaging/message_dispatch_service.py 9.09% 10 Missing :warning:
Additional details and impacted files
@@                      Coverage Diff                       @@
##           manual-task-dsr-scheduling    #6268      +/-   ##
==============================================================
+ Coverage                       84.66%   86.77%   +2.10%     
==============================================================
  Files                             444      444              
  Lines                           28395    28416      +21     
  Branches                         3158     3162       +4     
==============================================================
+ Hits                            24042    24658     +616     
+ Misses                           3624     3031     -593     
+ Partials                          729      727       -2     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Jun 25 '25 00:06 codecov[bot]

fides    Run #13052

Run Properties:  status check passed Passed #13052  •  git commit b016bc2f23: Email for external user invitation (#6268)
Project fides
Branch Review main
Run status status check passed Passed #13052
Run duration 00m 54s
Commit git commit b016bc2f23: Email for external user invitation (#6268)
Committer Adrian Galvan
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 5
View all changes introduced in this branch ↗︎

cypress[bot] avatar Jul 02 '25 17:07 cypress[bot]