Metadata polish + README roadmap update - #24
Merged
Conversation
Adds Open Graph/Twitter card metadata and a themeColor viewport entry
to the root layout, plus a title template ("%s | Vortex") so future
per-page titles compose cleanly with the site name. Adds
src/app/icon.svg (Next's App Router favicon convention) using the same
mark as VortexLogo — the app had no favicon at all before this.
Rewrites README.md: fixes the CI badge and every cross-repo link, which
pointed at the "vortex-protocol" org while this repo actually lives
under "stellar-vortex-protocol" (the badge was pointing at a workflow
that doesn't exist). Adds the /explore and /explore/[id] routes to the
pages table (missing since PR #10/#12), a "Features" section
summarizing what's actually implemented now (wallet, live data,
solvers, notifications, a11y), `npm test` to the scripts table, and
checks off the roadmap items this 20-PR pass completed.
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
1 participant
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.
Summary
themeColorviewport entry to the root layout, plus a title template ("%s | Vortex") so future per-page titles compose cleanly with the site name.src/app/icon.svg(Next's App Router favicon convention) using the same mark asVortexLogo— the app had no favicon at all before this.README.md:vortex-protocolorg while this repo actually lives understellar-vortex-protocol— the badge was pointing at a workflow that doesn't exist./exploreand/explore/[id]routes to the pages table (missing since PR feat: submit swaps end-to-end with Freighter signing #10/Merge pull request #11 from stellar-vortex-protocol/pr9-swap-submissi… #12).npm testto the scripts table.Test plan
npm run lintnpm test(139 total passing, no changes to test files in this PR)rm -rf .next && npm run build— confirms/icon.svgregisters as a routenpm run devserves/with a 200,/icon.svgwith a 200, and the<title>tag renders correctlyBranched off
main(PR #19 already merged). This is the last of the 20 PRs in this pass.