Skip to content

Raise helpful error if IOWebApp fails to mount#346

Merged
armanbilge merged 5 commits intoarmanbilge:mainfrom
gmkumar2005:main
Jan 29, 2024
Merged

Raise helpful error if IOWebApp fails to mount#346
armanbilge merged 5 commits intoarmanbilge:mainfrom
gmkumar2005:main

Conversation

@gmkumar2005
Copy link
Copy Markdown
Contributor

Error messages during initialization will help beginners falling into trouble.

@gmkumar2005
Copy link
Copy Markdown
Contributor Author

With this change IOWebapp will throw an error if proper div is not found in the html which is used to load the scalajs application.
s"Unable to mount Calico. Check if $rootElementId exists in html")

Comment thread calico/src/main/scala/calico/IOWebApp.scala Outdated
@gmkumar2005
Copy link
Copy Markdown
Contributor Author

Fixed the error message.

Comment thread calico/src/main/scala/calico/IOWebApp.scala Outdated
@armanbilge armanbilge changed the title RaiseError in app initialization Raise helpful error if IOWebApp fails to mount Jan 29, 2024
Copy link
Copy Markdown
Owner

@armanbilge armanbilge left a comment

Choose a reason for hiding this comment

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

Thanks a lot! I can't believe I left a raw map(_.get) there 😅

@armanbilge armanbilge merged commit 7703b48 into armanbilge:main Jan 29, 2024
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.

2 participants