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

Edit page error for geowidget #125

Draft
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

nickmoreton
Copy link

@nickmoreton nickmoreton commented May 17, 2024

We noticed some errors reported in logs on a site.

It turns out there's a small regression in our forks main branch following the latest merges here: torchbox-forks@1ab86c6

This adds checks back in to test for Point having a value before it's used for comparison.

The change could be added to: #124 but we'll shortly be updating our main branch and the change should then be available in our PR

@nickmoreton nickmoreton changed the title Fix/wagtail 60 value data Edit page error for geowidget May 17, 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