[New Tx Flow] Fix bugs and update 'restore footprint' & 'extend ttl' functinality#2006
Merged
jeesunikim merged 7 commits intofeature/new-tx-flowfrom Apr 6, 2026
Merged
Conversation
…time setState - Add explicit switch cases for destination/source_account in ClassicOperation so onChange receives a string (not ChangeEvent) matching the component's API - Move setBuildClassicXdr into useEffect to avoid setState during render - Ensure all hooks run before early returns to fix "more hooks" error - Fix duplicate React key in SignerSelector dropdown items Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…e removal, and ContractStorage restore - Re-run Operations init effect when store resets to empty (clear all fix) - Pass source_account onChange as plain string (remove fake event wrapper) - Gate validation error card on hasFormContent to avoid flash after clear - Use window.open directly for internal dashboard links (bypass sanitizeUrl) - Remove resource_fee from restore_footprint/extend_footprint_ttl params - Migrate ContractStorage handleRestore to useBuildFlowStore Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ntent guard - Remove Resource Fee field interactions from Soroban extend/restore tests - Update initial state test to reflect hidden errors on blank form - Update clear-all test to expect errors hidden after reset Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Preview is available here: |
Revert hasFormContent guard so required-field errors are always visible, helping users see what's needed upfront. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Preview is available here: |
quietbits
approved these changes
Apr 6, 2026
src/app/(sidebar)/transaction/build/components/SimulateStepContent.tsx
Outdated
Show resolved
Hide resolved
src/app/(sidebar)/transaction/build/components/SubmitStepContent.tsx
Outdated
Show resolved
Hide resolved
|
Preview is available here: |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Applied the new store to contract storage to enable link via session storage via url
Removed the old prepare transaction for restore footprint/extend footprint related operation
authModeto be enabled only forinvoke_contract_functionAlways show submit via RPC if it's a soroban op and rpc is available
Fix
View in Dashboardlink. The previousopenUrldidn't work well with messy urlrestore_footprint build step