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.

Sign up for GitHub

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

Jump to bottom

no 404 page in drep directory #928

Closed
m-i-k-e opened this issue May 1, 2024 · 4 comments · Fixed by #1018
Closed

no 404 page in drep directory #928

m-i-k-e opened this issue May 1, 2024 · 4 comments · Fixed by #1018
Assignees
@pmbinapps
Labels
🐛 Bug Something isn't working

Comments

@m-i-k-e
Copy link

m-i-k-e commented May 1, 2024

Context & versions

staging.sanchonet

Steps to reproduce

  1. Go to a Drep detail page
  2. Add any character to the end of the url
  3. hit return

Loom video

https://www.loom.com/share/709bf5a3f1ca44a997938f05ea93c90c?sid=8ae0da24-5e01-4048-87f0-18a5962a6cc4

Actual behavior

You go to a page with an endless spinner

Expected behavior

You should have a page not found or Drep not found error
@m-i-k-e m-i-k-e added the 🐛 Bug Something isn't working label May 1, 2024
@pmbinapps
Copy link

Valid bug - page not found is not shown, instead endless loader is shown.

@pmbinapps
Copy link

Now loader is shown:

Image

Request should return similary like GA not found.

@m-i-k-e
Copy link
Author

m-i-k-e commented May 13, 2024 •
edited
Loading

@pmbinapps @MSzalowski In the case where there is a missing DRep, this is the page: https://www.figma.com/design/mJTBiUofLfb74Xc3W2U59o/Voltaire---GOV-TOOL?m=auto&node-id=9347%3A308889&t=VewrRakbrZccwD9V-1

@MSzalowski MSzalowski removed their assignment May 13, 2024
@vercia vercia mentioned this issue May 14, 2024
Merged
5 tasks
@vercia vercia linked a pull request May 14, 2024 that will close this issue
fix/928-not-404-page-in-drep-directory #1018
Merged
5 tasks
@vercia vercia self-assigned this May 14, 2024
@MSzalowski MSzalowski mentioned this issue May 15, 2024
Merged
@pmbinapps
Copy link

OK - Drep - not found shown instead:

Image

@pmbinapps pmbinapps assigned pmbinapps and unassigned vercia May 15, 2024
MSzalowski added a commit that referenced this issue May 16, 2024
@MSzalowski
Merge pull request #1030 from IntersectMBO/develop
f307ebd 
Sync test with develop: #957, #1028, #928, #1019
@pmbinapps pmbinapps mentioned this issue May 16, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pmbinapps pmbinapps

Labels
🐛 Bug Something isn't working
Projects
Govtool all
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix/928-not-404-page-in-drep-directory IntersectMBO/govtool
4 participants
@m-i-k-e @vercia @MSzalowski @pmbinapps