Skip to content

fix(ui): clear KMS settings when toggling to Public privacy mode - #594

Merged
Tybravo merged 2 commits into
Tybravo:mainfrom
QueenColly:fix/435-privacy-toggle-state
Aug 31, 2026
Merged

fix(ui): clear KMS settings when toggling to Public privacy mode#594
Tybravo merged 2 commits into
Tybravo:mainfrom
QueenColly:fix/435-privacy-toggle-state

Conversation

@QueenColly

Copy link
Copy Markdown
Contributor

Summary
This PR resolves #435 by fixing a state inconsistency bug in the SPV Wizard's privacy step. Previously, switching from KMS back to Public left orphaned KMS data in the state. The onChange handler has been updated to explicitly nullify KMS-specific fields whenever the Public option is selected.

Changes Proposed

Updated SPVPrivacyStep.tsx to intercept the privacy mode toggle event.

Added state cleanup logic to wipe kmsProvider, kmsKeyId, and kmsRegion when switching to public.

Acceptance Criteria Checklist

[x] Clear KMS-specific settings if user switches back to Public.

[x] State remains consistent and clean without orphaned payload data.

Closes #435

@drips-wave

drips-wave Bot commented Aug 29, 2026

Copy link
Copy Markdown

@QueenColly Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Tybravo

Tybravo commented Aug 30, 2026

Copy link
Copy Markdown
Owner

@QueenColly

Please fix the CI Build and Check errors

Thanks

@Tybravo
Tybravo merged commit bbc89e5 into Tybravo:main Aug 31, 2026
1 check failed
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.

[Frontend] [Bug] Fix UI state inconsistency when toggling between Public and KMS in Wizard

2 participants