Skip to content

fix: bulk select normalization #7022

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

fix: bulk select normalization #7022

wants to merge 1 commit into from

Conversation

@speaker-ender
Copy link
Contributor

@speaker-ender speaker-ender commented Nov 24, 2025

Ticket []

Description Of Changes

Fixes a bugs when dealing with async data in the list and drawer view and starts to establish a pattern for normalizing data as nodes.

Code Changes

Steps to Confirm

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
  • UX feedback:
    • All UX related changes have been reviewed by a designer
    • No UX review needed
  • 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

vercel bot commented Nov 24, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
fides-plus-nightly Ready Ready Preview Comment Nov 26, 2025 3:59pm
1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
fides-privacy-center Ignored Ignored Nov 26, 2025 3:59pm

@speaker-ender speaker-ender force-pushed the fix/bulk-select-normalization--squashed branch 2 times, most recently from cc4a930 to 82ee727 Compare November 24, 2025 17:48
@speaker-ender speaker-ender force-pushed the fix/bulk-select-normalization--squashed branch from 82ee727 to 5f5ec21 Compare November 25, 2025 04:05
@speaker-ender
fix: bulk select normalization
1f262e2 
fix: bulk select normalization

wip: nearly there

chore: fixing naming update

test: adding normalization test

refactor: lots of small refinements

wip: some linting and adding new hook

refactors

chore: linting

refactor: names and interfaces

chore: linting

test: bulk list select

chore: linting

fix: rendering issues

fix: tests
@speaker-ender speaker-ender force-pushed the fix/bulk-select-normalization--squashed branch from 5f5ec21 to 1f262e2 Compare November 26, 2025 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

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

@speaker-ender