mapbox / mapbox/vtshaver

Option to keep all feature properties

Open
#46 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
44
Forks
7
PR merge metrics
No merged PRs in 30d

Description

https://github.com/mapbox/vtshaver/pull/20 landed the ability to have vtshaver automatically remove feature properties that are not needed for a given style.

However there are use cases (like querying) where some users might want feature properties to be kept.

A sophisticated solution for this would be to have vtshaver respond to metadata in the style that declares which properties to keep on which layers. However a more near-term solution I think would be to open up an option, exposed in the vtshaver CLI tool, that would disable all property shaving.

Contributor guide

Open the contributing guide

Research direction

Start by reading the implementation from pull request #20 and locating the vtshaver CLI entry point. Trace where feature-property shaving is enabled, then add the requested CLI option to disable it. Done means users can keep all feature properties while the existing shaving behavior remains unchanged when the option is not used.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
cli
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.