Skip to content

fix: improve error handling and re-routing#607

Merged
ferruhcihan merged 3 commits into
mainfrom
APL-938
Jul 8, 2025
Merged

fix: improve error handling and re-routing#607
ferruhcihan merged 3 commits into
mainfrom
APL-938

Conversation

@ferruhcihan
Copy link
Copy Markdown
Collaborator

📌 Summary

This PR fixes and improves error handling for clearing errors and re-routing, ensuring the desired path is preserved.

Ticket: https://jira.linode.com/browse/APL-938

🔍 Reviewer Notes

🧹 Checklist

  • Code is readable, maintainable, and robust.
  • Unit tests added/updated

Considerations

  • I have tested the changes in both light and dark mode.
  • I have considered the need for new unit tests.
  • I have tested the changes on a cluster.
  • I have included relevant documentation updates.
  • I have an approved Figma design or have reflected my changes in Figma
  • I have verified that the UI/UX is consistent in major browsers (e.g., Chrome, Firefox, Safari, Edge).
  • I have tested the changes for responsiveness in different screen resolutions.
  • I have tested expected error states and verified that the user is presented with informative error messages.
  • I have tested the feature with unusual or extreme inputs (e.g., very long strings, empty states, clicking a button multiple times quickly).

Copy link
Copy Markdown
Contributor

@CasLubbers CasLubbers left a comment

Choose a reason for hiding this comment

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

404 not found error still sticks for me when switching pages.

Copy link
Copy Markdown
Contributor

@CasLubbers CasLubbers left a comment

Choose a reason for hiding this comment

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

Works now

@ferruhcihan ferruhcihan merged commit 43617e7 into main Jul 8, 2025
6 checks passed
@ferruhcihan ferruhcihan deleted the APL-938 branch July 8, 2025 09:01
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.

4 participants