Skip to content

Add error handling#391

Merged
martinRenou merged 6 commits intogeojupyter:mainfrom
Meriem-BenIsmail:error-handling
Jan 31, 2025
Merged

Add error handling#391
martinRenou merged 6 commits intogeojupyter:mainfrom
Meriem-BenIsmail:error-handling

Conversation

@Meriem-BenIsmail
Copy link
Member

@Meriem-BenIsmail Meriem-BenIsmail commented Jan 24, 2025

Description

Resolves #327.

Checklist

  • PR has a descriptive title and content.
  • PR description contains references to any issues the PR resolves, e.g. Resolves #XXX.
  • PR has one of the labels: documentation, bug, enhancement, feature, maintenance
  • Checks are passing.
    Failing lint checks can be resolved with:
    • pre-commit run --all-files
    • jlpm run lint

📚 Documentation preview: https://jupytergis--391.org.readthedocs.build/en/391/
💡 JupyterLite preview: https://jupytergis--391.org.readthedocs.build/en/391/lite

@Meriem-BenIsmail Meriem-BenIsmail added the enhancement New feature or request label Jan 24, 2025
@github-actions
Copy link
Contributor

Binder 👈 Launch a Binder on branch Meriem-BenIsmail/jupytergis/error-handling

@github-actions
Copy link
Contributor

github-actions bot commented Jan 24, 2025

Integration tests report: appsharing.space

@Meriem-BenIsmail Meriem-BenIsmail marked this pull request as ready for review January 28, 2025 11:02
@Meriem-BenIsmail Meriem-BenIsmail marked this pull request as draft January 29, 2025 14:43
@Meriem-BenIsmail Meriem-BenIsmail force-pushed the error-handling branch 3 times, most recently from 451a03b to 7fcc020 Compare January 29, 2025 16:39
@Meriem-BenIsmail Meriem-BenIsmail marked this pull request as ready for review January 30, 2025 08:16
@martinRenou
Copy link
Member

@Meriem-BenIsmail if you rebase your PR, the CI should be green 🤞🏽

Copy link
Member

@martinRenou martinRenou left a comment

Choose a reason for hiding this comment

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

Neat! I have a couple of suggestions

@martinRenou
Copy link
Member

Triggering a rebuild with new readthedocs configuration, some readthedocs plugin seems to go in my way for testing the Lite deployment

Copy link
Member

@martinRenou martinRenou left a comment

Choose a reason for hiding this comment

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

Thanks!

@martinRenou martinRenou merged commit 4bd1d39 into geojupyter:main Jan 31, 2025
11 of 13 checks passed
HaudinFlorence pushed a commit to HaudinFlorence/jupytergis that referenced this pull request Jan 28, 2026
* added error handling

* modify error msg.

* store invalid layers+errors

* include error message in the dialog.

* compare error message.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Better error reporting

2 participants