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

Redirect to page not found not working on root route #257

Open
saivishwak opened this issue Sep 24, 2022 · 0 comments
Open

Redirect to page not found not working on root route #257

saivishwak opened this issue Sep 24, 2022 · 0 comments
Labels
bug Something isn't working

Comments

@saivishwak
Copy link

saivishwak commented Sep 24, 2022

Describe the bug
When we enter http://localhost:3005/asdfasdfasdfadf . This route is redirecting to /:JobID route rather than page not found.

Usually everyone tends to type something on the root route ("/") to navigate to the next page. With the existing approach this feels a like a bad ux experience. And also later when the app needs to have different routes which are not related to data, they can go to their route like /settings/my_profile , /users/:userID. This will be useful when the app grows.

To Reproduce
open localhost and enter random route like http://localhost:3005/asdfasdfasdfa

Some advices:

  • Please don't use customer data or custom paths.
  • Link to a gist would be appreciated.

Expected behavior
The route should be redirected to page not found

Vesion (please complete the following information):

  • OS: Windows (WSL2)
  • Azimuth version: 2.3.0

Additional context
Adding a screenshot of the issue

1

@saivishwak saivishwak added the bug Something isn't working label Sep 24, 2022
@gabegma gabegma added this to Azimuth Sep 26, 2022
@gabegma gabegma moved this to 🆕 New in Azimuth Sep 26, 2022
@gabegma gabegma moved this from 🆕 New to 🏗 In progress in Azimuth Sep 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: 🏗 In progress
Development

Successfully merging a pull request may close this issue.

1 participant