Skip to content

Feature/5576 wfs map ovelays#5670

Draft
robinmolen wants to merge 5 commits intomainfrom
feature/5576-wfs-map-ovelays
Draft

Feature/5576 wfs map ovelays#5670
robinmolen wants to merge 5 commits intomainfrom
feature/5576-wfs-map-ovelays

Conversation

@robinmolen
Copy link
Copy Markdown
Contributor

@robinmolen robinmolen commented Oct 2, 2025

Partly closes #5576

This PR relies on a new version of the Formio-builder, to support WFS tile layer configuration on map components. This should not be merged, until the Formio-builder has been updated and the dependency in this PR has been upgraded!

Changes

Added configuration for WFS tile layers, exposed the WFS tile layers to the formio-builder, and extended the email digest checking.

The implementation is mostly the same as the implementation for the WMS tile layers: using the same structure and naming for the model fields. The WFS tile layers can also be imported and exported through the admin configuration page.

The email digest for WMS has been altered to a more generic map overlay check. Both WMS and WFS tile layers are now checked and reported for problems.

Checklist

Check off the items that are completed or not relevant.

  • Impact on features

    • Checked copying a form
    • Checked import/export of a form
    • Config checks in the configuration overview admin page
    • Problem detection in the admin email digest is handled
  • Dockerfile/scripts

    • Updated the Dockerfile with the necessary scripts from the ./bin folder
  • Commit hygiene

    • Commit messages refer to the relevant Github issue
    • Commit messages explain the "why" of change, not the how

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 2, 2025

Codecov Report

❌ Patch coverage is 91.26214% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.98%. Comparing base (ac4c5b0) to head (2597156).
⚠️ Report is 28 commits behind head on master.

Files with missing lines Patch % Lines
src/openforms/emails/digest.py 87.30% 4 Missing and 4 partials ⚠️
src/openforms/config/models/map.py 93.33% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5670      +/-   ##
==========================================
- Coverage   97.00%   96.98%   -0.03%     
==========================================
  Files         835      835              
  Lines       30001    30078      +77     
  Branches     3895     3908      +13     
==========================================
+ Hits        29103    29171      +68     
- Misses        605      610       +5     
- Partials      293      297       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

WFS tile layers are similar to WMS tile layers, in the way that they both allow additional information to be presented in a map component. WFS takes it one step further, to allow the user to interact with the added visuals.

The WFS version of BAG has been added as default WFS tile layer, to quickly let users become familiar with the new functionality.

The WFS tile layers can be imported and exported from the WFS tile layers configuration page in the admin. This allows quick and easy sharing of WFS tile layers between OF instances.
Just like the background layers and WMS layers, this is exposed through a simple
JSON object dumped in the template context - there is no need to
add additional API endpoints.
To showcase the map component overlays, we need to know which url corresponds to the selected tile layer. By fetching the url before the component is shown in the form, ensure that we have the current and correct url.
Map component configuration can become invalid in multiple ways:
- tile layer source that updates its available layers/feature types
- tile layers that are removed in OF
- tile layer sources that are no-longer available

The email digest for WMS has been transformed into a generic map overlay check, which covers WMS and WFS. Internally WMS and WFS are still handled separately, because of their different XML structures and rules.
By using the namespaces that are actually set on the WMS and WFS tile layers, we can support more sources than just openGIS.

In addition, by checking the availability of the namespaces, we can target the xml content more accurately.
@robinmolen robinmolen force-pushed the feature/5576-wfs-map-ovelays branch from 27a25dc to 2597156 Compare October 7, 2025 15:02
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.

Kaartmateriaal - add WFS layer support

1 participant