Skip to content

Fix version pages showing a HTTP 500 error when the version does not exist#9615

Merged
andreslucena merged 3 commits intodecidim:developfrom
mainio:fix/9609
Sep 13, 2022
Merged

Fix version pages showing a HTTP 500 error when the version does not exist#9615
andreslucena merged 3 commits intodecidim:developfrom
mainio:fix/9609

Conversation

@ahukkanen
Copy link
Copy Markdown
Contributor

🎩 What? Why?

Show a 404 instead of 500 on the version pages when the version number does not exist.

📌 Related Issues

Testing

  • Go into one of the version pages (e.g. proposal versions)
  • Change the URL with a version number that doesn't exist

Copy link
Copy Markdown
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checked it locally, it works as expected 👍🏽

@andreslucena andreslucena merged commit 159f90d into decidim:develop Sep 13, 2022
@ahukkanen ahukkanen deleted the fix/9609 branch September 13, 2022 11:14
entantoencuanto added a commit that referenced this pull request Sep 13, 2022
* develop:
  Add missing character on code block (#9798)
  Fix hidden error messages on the registration form (#9625)
  Add documentation about configuring ActiveStorage / dynamic file uploads (#9777)
  Add documentation section about customizing cells (#9622)
  Fix hashtags not recognized at the beginning of the string (#9616)
  Fix version pages showing a HTTP 500 error when the version does not exist (#9615)
  Fix multitenant organizations stats cache (#9605)
  Prevent the account edit route through Devise (#9611)
  Fix iframe disabling producing invalid HTML (#9685)
  Fix import of images on spaces (#9779)
  Fix order of last activities (#9756)
  Fix leaking emails on admin user search controller (#9791)
  Ignore participatory spaces without models in meetings visible_for scope (#9790)
entantoencuanto added a commit that referenced this pull request Sep 15, 2022
* develop: (24 commits)
  Add develop index to the documentation (#9666)
  Fix initiatives components (#9633)
  Fix conference speaker avatars (#9643)
  Update `rokroskar/workflow-run-cleanup-action` GitHub action to v0.3.3 (#9750)
  Fix character counter for the WYSIWYG editor (#9680)
  Fix posting comments before the initial load has run (#9614)
  Fix parallel tests port in use (#9661)
  Split parallel test coverage reports into their own folders (#9686)
  Improve admin panel user experience regarding title links and order of actions (#9496)
  Fix title and description too long in initiatives spec sometimes (#9648)
  Fix API GraphiQL system spec with newer ChromeDriver (#9642)
  Add missing character on code block (#9798)
  Fix hidden error messages on the registration form (#9625)
  Add documentation about configuring ActiveStorage / dynamic file uploads (#9777)
  Add documentation section about customizing cells (#9622)
  Fix hashtags not recognized at the beginning of the string (#9616)
  Fix version pages showing a HTTP 500 error when the version does not exist (#9615)
  Fix multitenant organizations stats cache (#9605)
  Prevent the account edit route through Devise (#9611)
  Fix iframe disabling producing invalid HTML (#9685)
  ...
eliegaboriau pushed a commit to eliegaboriau/decidim that referenced this pull request Oct 25, 2022
…exist (decidim#9615)

* Show a 404 when the version does not exist

* Add specs for testing the versions controllers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error 500 in the versions

2 participants