Skip to content

docs: add contributor guide for adding a new page route (closes #426)#472

Merged
Chucks1093 merged 1 commit into
accesslayerorg:devfrom
LaPoshBaby:docs/adding-page-routes-guide
Jun 26, 2026
Merged

docs: add contributor guide for adding a new page route (closes #426)#472
Chucks1093 merged 1 commit into
accesslayerorg:devfrom
LaPoshBaby:docs/adding-page-routes-guide

Conversation

@LaPoshBaby

Copy link
Copy Markdown
Contributor

Summary

  • Adds docs/adding-page-routes.md: a contributor guide covering where routes are registered in src/App.tsx, the file naming convention for page components (PascalCase + Page suffix, default export, in src/pages/), the recommended pattern for auth-protected routes using a RequireAuth wrapper around useAccount().isConnected from wagmi, and a four-step worked example adding a public AboutPage at /about.
  • Adds a one-line pointer in CONTRIBUTING.md under "Frontend conventions" so contributors can find the guide.

Testing

  • pnpm lint passed (only one unrelated pre-existing warning in CreatorCard.tsx:119).
  • pnpm build not run; this is a docs-only change and no source code was modified.

Checklist

Closes accesslayerorg#426.

Adds docs/adding-page-routes.md covering where routes are registered in src/App.tsx, the file naming convention for page components, the recommended pattern for auth-protected routes (with a RequireAuth wrapper example), and a four-step worked example adding a public AboutPage at /about. Adds a one-line pointer in CONTRIBUTING.md under Frontend conventions so contributors can find the guide.
@drips-wave

drips-wave Bot commented Jun 26, 2026

Copy link
Copy Markdown

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

@Chucks1093 Chucks1093 merged commit 203907c into accesslayerorg:dev Jun 26, 2026
1 check 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

Development

Successfully merging this pull request may close these issues.

Add docs for how to add a new page route to the client

2 participants