Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Routing: Dashboard pages do (some) routing manually #566

Open
Danziger opened this issue Dec 4, 2024 · 0 comments
Open

Routing: Dashboard pages do (some) routing manually #566

Danziger opened this issue Dec 4, 2024 · 0 comments
Labels

Comments

@Danziger
Copy link
Contributor

Danziger commented Dec 4, 2024

ArConnect/src/routes/dashboard/index.tsx still performs routing manually in the center panel. That should be replace with <Routes>, just like it's been done for the third panel in #556.

This change should also allow the removal all usages of wouter's useRoute() in Dashboard views. The params should instead come from the view props (props.params). That requires some views to be refactored to use the router <Routes>.

@Danziger Danziger added the good first issue Good for newcomers label Dec 4, 2024
@Danziger Danziger changed the title Auth: Dashboard pages do (some) routing manually Routing: Dashboard pages do (some) routing manually Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant