Skip to content

fix(#567,#568,#561,#562): button size prop, navbar active link, dynamic imports, SEP-0010 auth - #707

Merged
Kingsman-99 merged 1 commit into
Stellar-split:mainfrom
marshall8000:feature/issues-561-562-567-568
Aug 30, 2026
Merged

fix(#567,#568,#561,#562): button size prop, navbar active link, dynamic imports, SEP-0010 auth#707
Kingsman-99 merged 1 commit into
Stellar-split:mainfrom
marshall8000:feature/issues-561-562-567-568

Conversation

@marshall8000

Copy link
Copy Markdown
Contributor

#567 — Button size prop (sm/md/lg)

  • Button component already had complete size prop implementation
  • Verified: size prop typed as 'sm' | 'md' | 'lg' with default 'md'
  • Each size maps to distinct Tailwind padding and text-size classes
  • All existing usages without size prop continue to render identically

#568 — Navbar active link with prefix matching

  • Updated isActive() to use pathname.startsWith() for prefix matching
  • Added /invoice nav item so /invoice/* routes highlight correctly
  • Exact match preserved for root-level routes
  • No link highlighted on pages that don't match any nav item

#561 — Dynamic imports and route-level code splitting

  • NotesEditor: added NotesEditor.dynamic.tsx with next/dynamic + ssr:false
  • TrendChart: created new TrendChart component + TrendChart.dynamic.tsx
  • transactionBuilder: new src/lib/stellar/transactionBuilder.ts loaded dynamically in invoice sign page to exclude stellar-sdk from initial bundle
  • next.config.js: fixed bundle analyzer (ANALYZE=true gates report), fixed withPWA config, fixed syntax error, fixed env var typo

#562 — SEP-0010 Web Authentication

  • GET /api/auth/challenge: returns signed challenge tx (5 min TTL)
  • POST /api/auth/token: verifies signed tx, issues 24h JWT in httpOnly cookie
  • src/lib/auth/withAuth.ts: route handler wrapper requiring valid JWT (401 if absent/expired)
  • src/hooks/useSep10Auth.ts: challenge->sign->token flow hook with auto-retry on 401
  • WalletContext: extended with sep10 auth state, auto-authenticates on wallet connect

Misc: cleaned up .gitignore (removed duplicates, added all test artifact patterns)

closes #567
closes #568
closes #561
closes #562

…it#562): button size prop, navbar active link, dynamic imports, SEP-0010 auth

Stellar-split#567 — Button size prop (sm/md/lg)
- Button component already had complete size prop implementation
- Verified: size prop typed as 'sm' | 'md' | 'lg' with default 'md'
- Each size maps to distinct Tailwind padding and text-size classes
- All existing usages without size prop continue to render identically

Stellar-split#568 — Navbar active link with prefix matching
- Updated isActive() to use pathname.startsWith() for prefix matching
- Added /invoice nav item so /invoice/* routes highlight correctly
- Exact match preserved for root-level routes
- No link highlighted on pages that don't match any nav item

Stellar-split#561 — Dynamic imports and route-level code splitting
- NotesEditor: added NotesEditor.dynamic.tsx with next/dynamic + ssr:false
- TrendChart: created new TrendChart component + TrendChart.dynamic.tsx
- transactionBuilder: new src/lib/stellar/transactionBuilder.ts loaded
  dynamically in invoice sign page to exclude stellar-sdk from initial bundle
- next.config.js: fixed bundle analyzer (ANALYZE=true gates report),
  fixed withPWA config, fixed syntax error, fixed env var typo

Stellar-split#562 — SEP-0010 Web Authentication
- GET /api/auth/challenge: returns signed challenge tx (5 min TTL)
- POST /api/auth/token: verifies signed tx, issues 24h JWT in httpOnly cookie
- src/lib/auth/withAuth.ts: route handler wrapper requiring valid JWT (401 if absent/expired)
- src/hooks/useSep10Auth.ts: challenge->sign->token flow hook with auto-retry on 401
- WalletContext: extended with sep10 auth state, auto-authenticates on wallet connect

Misc: cleaned up .gitignore (removed duplicates, added all test artifact patterns)
@drips-wave

drips-wave Bot commented Aug 29, 2026

Copy link
Copy Markdown

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

@vercel

vercel Bot commented Aug 29, 2026

Copy link
Copy Markdown

@marshall8000 is attempting to deploy a commit to the kingsman-99's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Kingsman-99
Kingsman-99 merged commit 398b978 into Stellar-split:main Aug 30, 2026
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants