Skip to content

feat(featureserver): add default value for indexes on feature layer - #1210

Merged
anders-kane merged 2 commits into
masterfrom
layer-metadata-indexes-default
Mar 31, 2026
Merged

feat(featureserver): add default value for indexes on feature layer #1210
anders-kane merged 2 commits into
masterfrom
layer-metadata-indexes-default

Conversation

@anders-kane

Copy link
Copy Markdown
Contributor

Add empty array default value for indexes on layer metadata, this is needed for the feature layer output from koop to meet the validation checks imposed during feature service create flow in ArcGIS online.

@anders-kane
anders-kane requested a review from sansth1010 March 31, 2026 19:31
@changeset-bot

changeset-bot Bot commented Mar 31, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: ff10221

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@koopjs/featureserver Minor
@koopjs/output-geoservices Patch
@koopjs/koop-core Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions

Copy link
Copy Markdown
Contributor

Coverage Report (change vs master)

File Path Statements Branches Functions Lines
packages/featureserver/src/metadata-defaults.js 100.0 green
vs
100.0 green
100.0 green
vs
100.0 green
100.0 green
vs
100.0 green
100.0 green
vs
100.0 green
packages/featureserver/src/helpers/feature-layer-metadata.js 100.0 green
vs
100.0 green
100.0 green
vs
100.0 green
100.0 green
vs
100.0 green
100.0 green
vs
100.0 green
packages/featureserver/src/helpers/table-layer-metadata.js 98.7 yellowGreen
vs
98.7 yellowGreen
89.8 yellow
vs
89.8 yellow
100.0 green
vs
100.0 green
98.7 yellowGreen
vs
98.7 yellowGreen

@anders-kane
anders-kane merged commit 0b9805d into master Mar 31, 2026
7 checks passed
@anders-kane
anders-kane deleted the layer-metadata-indexes-default branch March 31, 2026 20:01
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