Skip to content

HA-628 - Update the Datahub Permissions section to include required permissions from Datahub#6052

Merged
andres-torres-marroquin merged 3 commits intomainfrom
andres/HA-628
Apr 16, 2025
Merged

HA-628 - Update the Datahub Permissions section to include required permissions from Datahub#6052
andres-torres-marroquin merged 3 commits intomainfrom
andres/HA-628

Conversation

@andres-torres-marroquin
Copy link
Copy Markdown
Contributor

@andres-torres-marroquin andres-torres-marroquin commented Apr 16, 2025 •
edited by atlassian Bot
Loading

Closes HA-628

Description Of Changes

Update the Datahub Permissions section to include required permissions from Datahub

Code Changes

  • list your code changes here

Steps to Confirm

  1. Open Integrations
  2. Add Integration
  3. Datahub
  4. Details
  5. Expand "Show more"

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
  • Followup issues:
    • Followup issues created (include link)
    • 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 Apr 16, 2025 •
edited
Loading

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 Apr 16, 2025 8:40pm
fides-privacy-center ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 16, 2025 8:40pm

@andres-torres-marroquin andres-torres-marroquin merged commit 567f008 into main Apr 16, 2025
17 checks passed
@andres-torres-marroquin andres-torres-marroquin deleted the andres/HA-628 branch April 16, 2025 20:42
@cypress
Copy link
Copy Markdown

cypress Bot commented Apr 16, 2025

fides    Run #12827

Run Properties:  status check passed Passed #12827  •  git commit 567f008ed6: HA-628 - Update the Datahub Permissions section to include required permissions ...
Project fides
Branch Review main
Run status status check passed Passed #12827
Run duration 00m 52s
Commit git commit 567f008ed6: HA-628 - Update the Datahub Permissions section to include required permissions ...
Committer Andrés Torres Marroquín
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

Reviewers

@vcruces vcruces vcruces approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@andres-torres-marroquin @vcruces