Skip to content

fix(frontend)!: independent wallet store for each tool#610

Merged
DarianM merged 22 commits intomainfrom
auto_load_profiles_wa
Mar 16, 2026
Merged

fix(frontend)!: independent wallet store for each tool#610
DarianM merged 22 commits intomainfrom
auto_load_profiles_wa

Conversation

@DarianM
Copy link
Copy Markdown
Member

@DarianM DarianM commented Mar 11, 2026

Closes #603
Closes #319

@DarianM DarianM changed the base branch from main to reset_wallet_state March 11, 2026 10:03
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 11, 2026

Deployment results

Worker Alias URL Outcome
API - staging success
CDN - staging success
App - 832107dd success

Logs #23147387446

Base automatically changed from reset_wallet_state to main March 11, 2026 12:51
@DarianM DarianM changed the title fix(frontend): auto load tool profiles when wallet is already connected fix(frontend): independent wallet store for each tool Mar 12, 2026
resetProfiles() {
for (const actions of [bannerActions, widgetActions, offerwallActions]) {
actions.resetProfiles()
resetToolProfiles() {
Copy link
Copy Markdown
Member Author

@DarianM DarianM Mar 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO: resetToolProfiles, setToolProfiles, getToolProfiles do not belong into toolStore anymore (toolStore owns global UI state now, file name is deceiving).
Maybe move into a useToolProfiles hook?

@DarianM DarianM marked this pull request as ready for review March 13, 2026 12:48
return {
setWalletConnected(connected: boolean) {
wallet.isWalletConnected = connected
wallet.walletConnectStep = connected ? 'filled' : 'unfilled'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need this filled state? Can we just use isWalletConnected directly for steps as well?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will do this as a follow-up. true, we can just use isWalletConnected

@sidvishnoi sidvishnoi changed the title fix(frontend): independent wallet store for each tool fix(frontend)!: independent wallet store for each tool Mar 16, 2026
@DarianM DarianM merged commit 998623e into main Mar 16, 2026
5 of 6 checks passed
@DarianM DarianM deleted the auto_load_profiles_wa branch March 16, 2026 14:04
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.

[Bug] Each tool should be able to switch independently between different wallet addresses [BUG] Some config is retained between Widget & Banner

2 participants