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

[Bug]: server-url related errors are not surfaced #3784

Closed
2 tasks done
MatissJanis opened this issue Nov 4, 2024 · 4 comments · Fixed by #3837
Closed
2 tasks done

[Bug]: server-url related errors are not surfaced #3784

MatissJanis opened this issue Nov 4, 2024 · 4 comments · Fixed by #3837
Labels
bug Something isn't working regression This bug appeared after a recent release and needs to be fixed before the next release user interface Related to the user interface
Milestone

Comments

@MatissJanis
Copy link
Member

Verified issue does not already exist?

  • I have searched and found no existing issue
  • I will be providing steps how to reproduce the bug (in most cases this will also mean uploading a demo budget file)

What happened?

Screen.Recording.2024-11-04.at.18.13.15.mov

Where are you hosting Actual?

None

What browsers are you seeing the problem on?

No response

Operating System

None

@MatissJanis MatissJanis added bug Something isn't working help wanted Extra attention is needed user interface Related to the user interface regression This bug appeared after a recent release and needs to be fixed before the next release labels Nov 4, 2024
@shb9019
Copy link
Contributor

shb9019 commented Nov 9, 2024

Hi @MatissJanis, I'd love to take this up if no one's already working on it! I believe this regression was caused due to #3736, where we're silently suppressing the URL validation error. This behavior was alluded to in #3736 (review).

@MatissJanis
Copy link
Member Author

Feel free to send a PR for this. :)

@MatissJanis MatissJanis added this to the v24.12.0 milestone Nov 14, 2024
@MatissJanis
Copy link
Member Author

cc @MikesGlitch as an FYI

@MikesGlitch
Copy link
Contributor

MikesGlitch commented Nov 14, 2024

Oops. 😊

Yeah, I added that to fix a crash state where the user somehow had an incorrect server URL (I'm not sure how they got to that state). The joinURL function failed and they got a crashed state with no way to fix it. I think it's an edge case - but would be good if any fix could also solve that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working regression This bug appeared after a recent release and needs to be fixed before the next release user interface Related to the user interface
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants