Skip to content

Use all geojson locations for reference checks#28

Merged
binh-dam-ibigroup merged 24 commits into
dev-flexfrom
collect-geojson-ids
Jul 15, 2026
Merged

Use all geojson locations for reference checks#28
binh-dam-ibigroup merged 24 commits into
dev-flexfrom
collect-geojson-ids

Conversation

@binh-dam-ibigroup

@binh-dam-ibigroup binh-dam-ibigroup commented Jul 8, 2026

Copy link
Copy Markdown

Checklist

  • Appropriate branch selected => This PR is against dev-flex (all PRs must first be merged to dev before they can be merged to master)
  • Any modified or new methods or classes have helpful JavaDoc and code is thoroughly commented
  • The description lists all applicable issues this PR seeks to resolve
  • [na] The description lists any configuration setting(s) that differ from the default settings
  • All tests and CI builds passing

Description

This PR improves flex validation by removing incorrectly raised referential integrity errors for geojson locations of unsupported types, and by removing duplicate errors of unsupported geometry types.

@binh-dam-ibigroup
binh-dam-ibigroup marked this pull request as ready for review July 8, 2026 20:31

@br648 br648 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Good update. All unit tests pass.

@br648 br648 assigned binh-dam-ibigroup and unassigned br648 Jul 13, 2026

@josh-willis-arcadis josh-willis-arcadis left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM!

@binh-dam-ibigroup
binh-dam-ibigroup merged commit e254090 into dev-flex Jul 15, 2026
2 checks passed
@binh-dam-ibigroup
binh-dam-ibigroup deleted the collect-geojson-ids branch July 15, 2026 16:48
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.

3 participants