Skip to content

Add "view" detail links to action center success toasts#6113

Merged
jpople merged 11 commits intomainfrom
jpople/ha-405/toast-view-links
May 6, 2025
Merged

Add "view" detail links to action center success toasts#6113
jpople merged 11 commits intomainfrom
jpople/ha-405/toast-view-links

Conversation

@jpople
Copy link
Copy Markdown
Contributor

@jpople jpople commented May 5, 2025

Closes HA-405

Description Of Changes

Adds "View" links to success toasts on action center operations.

Screenshot 2025-05-05 at 14 38 19

Toasts link to the system an asset has been promoted to on promotion ("Add") and to the "Ignored" tab on the appropriate table on ignore.

Steps to Confirm

Run a Fidesplus backend on branch HA-636; otherwise system keys won't be present to be linked to

  1. Populate some web monitor results
  2. In uncategorized assets table, assign a system to an asset
  3. Click "add"
  4. Success toast should have link to that system's asset table
  5. In a categorized asset table, click "add" on an asset
  6. Should have link to appropriate system's asset table
  7. Click "ignore" on an asset
  8. Link should keep you on the same table but show "ignored" tab
  9. Select multiple assets with the same system in the table and select "Add" from actions
  10. Should have a link to that system's page
  11. Select multiple assets with different systems in the table and select "Add" from actions
  12. Should have a link to the main system inventory

Pre-Merge Checklist

  • Issue requirements met
  • All CI pipelines succeeded
  • CHANGELOG.md updated
    • Add a db-migration This indicates that a change includes a database migration label to the entry if your change includes a DB migration
    • Add a high-risk This issue suggests changes that have a high-probability of breaking existing code 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

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 5, 2025

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

Name Status Preview Comments Updated (UTC)
fides-plus-nightly ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 6, 2025 6:19pm
fides-privacy-center ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 6, 2025 6:19pm

Your system has been added to your data map. You can{" "}
<ToastLink onClick={onViewDatamap}>view it now</ToastLink> and come back
to finish this setup when you’re ready. Or you can progress to{" "}
<ToastLink onClick={onViewDatamap}>view it now</ToastLink> or{" "}
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated the styling on this to keep it consistent with the new toast links and because it had been bothering me, just tightening up this phrasing too.

Copy link
Copy Markdown
Contributor

@andres-torres-marroquin andres-torres-marroquin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works as expected.

@jpople jpople merged commit 2b1be4e into main May 6, 2025
17 checks passed
@jpople jpople deleted the jpople/ha-405/toast-view-links branch May 6, 2025 18:59
@cypress
Copy link
Copy Markdown

cypress Bot commented May 6, 2025

fides    Run #12897

Run Properties:  status check passed Passed #12897  •  git commit 2b1be4e18f: Add "view" detail links to action center success toasts (#6113)
Project fides
Branch Review main
Run status status check passed Passed #12897
Run duration 00m 57s
Commit git commit 2b1be4e18f: Add "view" detail links to action center success toasts (#6113)
Committer jpople
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 ↗︎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants