Merge from staging #742
Merged
Merge from staging #742
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The one-time Django→Prisma data migration is complete on all environments. This removes: - Data migration scripts (prisma/scripts/01-04*.ts) - Helper scripts (check-user.ts, legacy-team-cleanup.ts) - Migration notification data (duplicate-user-notifications.json) - tsx, postgres, dotenv dependencies from Docker image - Complex migration logic from docker-entrypoint.sh The entrypoint now simply runs `prisma migrate deploy` and starts the server.
Bug #1: After detaching children to sandbox, orphaned elements weren't showing in the reattach dialog until page reload. Fixed by updating handleDetachAndDelete to populate orphanedElements state immediately after detaching children (including nested subtrees). Bug #2: After reattaching an element with children, the children weren't rendering on the canvas until page reload. Fixed by replacing client-side state manipulation with server refetch approach (refetchCaseData) that fetches the complete tree with all children. Removed ~270 lines of unused client-side state manipulation functions that were replaced by the simpler server refetch approach.
…pdate The reset identifiers API was successfully updating the database but the UI wasn't refreshing because no SSE event was emitted. Added emitSSEEvent() call after successful update. Note: userId is intentionally not passed to emitSSEEvent() so the triggering user also receives the event (the frontend doesn't update optimistically for this operation). Closes: AssurancePlatform-dov
JWT-only auth has been stable for 30+ days. Remove the RefreshToken infrastructure that is no longer used: - Delete lib/auth/refresh-token-service.ts and lib/auth/feature-flags.ts - Simplify validate-session.ts to use getServerSession directly - Remove session token fields from NextAuth types - Remove RefreshToken model from Prisma schema - Update middleware to check only session.user.id - Update auth-options.ts to remove token creation code - Update client components to pass empty string for token parameter - Update test files to remove session token expectations Database migration included to drop refresh_tokens table.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.