Skip to content

feat: mount walletstatus in a shared appheader and remove dead connect button#382

Open
Hilary017 wants to merge 4 commits into
Liquifact:mainfrom
Hilary017:main
Open

feat: mount walletstatus in a shared appheader and remove dead connect button#382
Hilary017 wants to merge 4 commits into
Liquifact:mainfrom
Hilary017:main

Conversation

@Hilary017

@Hilary017 Hilary017 commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Closes #81


Summary

  • I implemented this feature using the already existing navmenu and walletstatus components. Making the walletstatus component available in all pages and also the back button to navigate back to the homepage when outside of the homepage.

Closes #

Validation

  • npm run lint
  • npm test
  • npm run build

Accessibility

  • Interactive controls have accessible names
  • Keyboard and focus-visible behavior were preserved
  • New or changed UI has relevant Jest/RTL or jest-axe coverage

Security

  • No secrets, wallet keys, .env files, or generated artifacts are included
  • API or wallet trust-boundary changes are explained

@drips-wave

drips-wave Bot commented Jun 30, 2026

Copy link
Copy Markdown

@Hilary017 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

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.

Mount the WalletStatus state machine in the app header and remove the dead Connect Wallet button

1 participant