feat: add route progress, PWA manifest, and wire notifications - #246
Open
overprodigy wants to merge 4 commits into
Open
feat: add route progress, PWA manifest, and wire notifications#246overprodigy wants to merge 4 commits into
overprodigy wants to merge 4 commits into
Conversation
Mount RouteProgressBar component in App.tsx to display a thin progress bar during route transitions. Respects prefers-reduced-motion for accessibility. Resolves TevaLabs#189
Create manifest.json with complete PWA configuration including app name, icons at common mobile sizes, theme colors, and display settings. Ensures Lighthouse PWA manifest checks pass. Resolves TevaLabs#187
Replace hardcoded 'user' placeholder with actual user publicKey from wallet store. Subscribe to notifications channel using the authenticated user's Stellar address. Re-subscribes when user connects or disconnects wallet. Resolves TevaLabs#184
|
@overprodigy is attempting to deploy a commit to the josephchimebuka's projects Team on Vercel. A member of the Team first needs to authorize it. |
Contributor
|
@overprodigy Can u fix your PR? |
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.
What
Add route progress indicator, PWA manifest for installability, and wire notifications to authenticated users.
Issues Resolved
Closes #189
Closes #187
Closes #184
Closes #186
Changes
#189 - Add route progress indicator for multi-page navigation
Mount RouteProgressBar component that displays a thin animated progress bar on route transitions, respects prefers-reduced-motion for accessibility.
#187 - Add web app manifest and installable PWA icons
Create manifest.json with complete PWA configuration including app name, icons at common mobile sizes (64x64, 192x192, 256x256, 512x512), theme colors, and display settings.
#184 - Wire NotificationsBell to authenticated user ID
Replace hardcoded 'user' placeholder with actual user publicKey from wallet store, ensuring notifications are routed to the correct authenticated user with re-subscription on wallet connect/disconnect.
#186 - Add Open Graph and Twitter card meta tags
Social preview meta tags and branding assets are already configured in index.html with proper og:image and twitter:image references to social-preview.png.