feat: Add PWA support and responsive UI improvements to vault-ui - #722
Merged
Abidoyesimze merged 1 commit intoAug 29, 2026
Merged
Conversation
|
@francisprecious061 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! 🚀 |
orebams
pushed a commit
to orebams/NeuroWealth-Smartcontract
that referenced
this pull request
Aug 29, 2026
Split out of PortfolioDashboard in a prior refactor but never adopted — the parent still renders the allocation panel inline, and this component's hardcoded English copy doesn't route through the app's i18n system (useI18n/AppMessages) the way PortfolioDashboard does, so wiring it in as-is would drop translations. Deleting rather than migrating avoids that regression. part of Neurowealth#722
orebams
pushed a commit
to orebams/NeuroWealth-Smartcontract
that referenced
this pull request
Aug 29, 2026
Same unwired-refactor situation as AllocationSection: split out of PortfolioDashboard but never adopted, and its copy is hardcoded English rather than i18n-driven like the inline panel it duplicates. part of Neurowealth#722
orebams
pushed a commit
to orebams/NeuroWealth-Smartcontract
that referenced
this pull request
Aug 29, 2026
Third of the three split-out panels that PortfolioDashboard never adopted; it still builds and renders its own summary cards inline. closes Neurowealth#722
orebams
pushed a commit
to orebams/NeuroWealth-Smartcontract
that referenced
this pull request
Aug 29, 2026
Its only consumer was the now-deleted AllocationSection component. part of Neurowealth#722
orebams
pushed a commit
to orebams/NeuroWealth-Smartcontract
that referenced
this pull request
Aug 29, 2026
closes Neurowealth#722 closes Neurowealth#723 closes Neurowealth#719 closes Neurowealth#726
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.
closes #663
#Summary
Describe the purpose of this PR and the changes included.
Checklist
CHANGELOG.mdunder[Unreleased]for any contract behavior, event, or error-code changes.get_version(), the new version number is noted inCHANGELOG.md.Versionstorage value.EVENTS.md.SECURITY.mdand added a corresponding test intest_pause.rs.