Skip to content

Default withAttrLayers to true for GeoJsonFolder#141

Merged
josephbirkner merged 3 commits intorelease/2026.1.0from
fix/140-withAttrLayers-default
Feb 3, 2026
Merged

Default withAttrLayers to true for GeoJsonFolder#141
josephbirkner merged 3 commits intorelease/2026.1.0from
fix/140-withAttrLayers-default

Conversation

@MisterGC
Copy link
Contributor

Change the withAttrLayers default from false to true so nested GeoJSON properties are converted to attribute layers by default.

  • Previously, nested object properties were silently dropped unless explicitly configured
  • Updated config schema with description and default value
  • Updated docs to clearly state the default

Fixes #140

Copy link
Contributor

@josephbirkner josephbirkner left a comment

Choose a reason for hiding this comment

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

πŸ‘

Add optional mapId config for GeoJsonFolder
@josephbirkner josephbirkner changed the base branch from main to release/2026.1.0 February 3, 2026 16:24
@josephbirkner josephbirkner merged commit c2a8f75 into release/2026.1.0 Feb 3, 2026
5 of 17 checks passed
@josephbirkner josephbirkner deleted the fix/140-withAttrLayers-default branch February 3, 2026 16:24
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 3, 2026

@github-actions
Copy link

github-actions bot commented Feb 3, 2026

Package Line Rate Branch Rate Health
libs.geojsonsource.include.geojsonsource 100% 50% βœ”
libs.geojsonsource.src 76% 43% βž–
libs.gridsource.include.gridsource 0% 0% ❌
libs.gridsource.src 0% 0% ❌
libs.http-datasource.src 57% 30% βž–
libs.http-service.include.mapget.http-service 100% 100% βœ”
libs.http-service.src 58% 20% βž–
libs.logging.include.mapget 8% 4% ❌
libs.model.include.mapget.model 54% 32% βž–
libs.model.src 62% 33% βž–
libs.service.include.mapget.service 40% 0% ❌
libs.service.src 78% 41% βž–
Summary 57% (4176 / 7343) 28% (4163 / 14628) βž–

Minimum allowed line rate is 50%

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.

GeoJsonSource: withAttrLayers defaults to false, silently dropping nested properties

2 participants