Skip to content

Improve error handling - #92

Merged
jernestmyers merged 3 commits into
mainfrom
improve-error-handling
Oct 28, 2026
Merged

jernestmyers merged 3 commits into
mainfrom
improve-error-handling

Conversation

@jernestmyers

Copy link
Copy Markdown
Collaborator

Does two main things:

  1. Refactors how Aladin is loaded and how its errors are handled. Now, loading the Aladin script is non-blocking. If Aladin is done, the same overlay as the "lost context" overlay is shown with a message and a "retry" button. Also, the lightcurve popup overlay has been adjusted to render about the overlay so that an Aladin error doesn't disrupt the lightcurve preview functionality on the main page.
  2. Refactors error handling regarding authentication. Makes it more clear when an error loading sources is related to authentication, and gives users instructions to log in or contact us (stubbed with the same environment variable for the "contact us" link in the bottom left footer.

I guess there's a third thing: there was a bug with the VITE_ORGANIZATION_NAME= env variable that prevented its use. It's been fixed.

@jernestmyers
jernestmyers merged commit 724615f into main Oct 28, 2026
1 check passed
@jernestmyers
jernestmyers deleted the improve-error-handling branch October 28, 2026 18:20
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.

1 participant