Skip to content

Add PLSS vector tile layer to overlay selector#254

Merged
stdavis merged 2 commits intomainfrom
copilot/add-plss-vector-tile-overlay
Apr 14, 2026
Merged

Add PLSS vector tile layer to overlay selector#254
stdavis merged 2 commits intomainfrom
copilot/add-plss-vector-tile-overlay

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 14, 2026

This adds the Utah PLSS vector tile service to the map’s overlay options so users can toggle it alongside parcel-related context layers. The change keeps the existing overlay behavior intact while exposing PLSS directly in the layer selector.

  • Overlay configuration

    • Added the PLSS VectorTileServer URL to app config.
  • Layer selector

    • Added a new PLSS overlay entry to the map layer menu.
    • Kept the existing Land Ownership overlay and appended PLSS as an additional operational layer.
    • Defined the new PLSS overlay inline in MapContainer per review feedback.
operationalLayers: [
  'Land Ownership',
  {
    label: 'PLSS',
    function: () =>
      new VectorTileLayer({
        url: config.plssVectorTileService,
      }),
  },
],
  • UI reference
    • Screenshot for review:

Copilot AI linked an issue Apr 14, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Add PLSS Vector Tile service as an Overlay option Add PLSS vector tile layer to overlay selector Apr 14, 2026
Copilot AI requested a review from stdavis April 14, 2026 19:03
Agent-Logs-Url: https://github.com/agrc/parcels/sessions/332dab33-96f3-4bb8-9c55-6771ab3f315c

Co-authored-by: stdavis <1326248+stdavis@users.noreply.github.com>
@stdavis stdavis force-pushed the copilot/add-plss-vector-tile-overlay branch from c45e2d8 to 176dd29 Compare April 14, 2026 19:10
Copy link
Copy Markdown
Member

@stdavis stdavis left a comment

Choose a reason for hiding this comment

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

The separate file and unit tests are overkill. Just add the new layer inline. Keep the URL in the config.

Agent-Logs-Url: https://github.com/agrc/parcels/sessions/27a39afc-4807-460b-a0a9-6a21fcbdbd3c

Co-authored-by: stdavis <1326248+stdavis@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 14, 2026

The separate file and unit tests are overkill. Just add the new layer inline. Keep the URL in the config.

Addressed in 1ed38b7. The PLSS layer is now defined inline in MapContainer, the URL stays in config, and the extra helper/test files were removed.

Screenshot: PLSS overlay option

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 14, 2026

Visit the preview URL for this PR (updated for commit 070c001):
Firebase Preview: https://ut-dts-agrc-parcels-dev--copilot-add-plss-vector-tile-vlo7snnm.web.app
(expires "2026-04-28T19:23:44.703391925Z" )

Copilot AI requested a review from stdavis April 14, 2026 19:16
Copilot AI deployed to preview April 14, 2026 19:18 Active
@stdavis stdavis marked this pull request as ready for review April 14, 2026 19:18
@stdavis stdavis enabled auto-merge (rebase) April 14, 2026 19:18
@stdavis stdavis disabled auto-merge April 14, 2026 19:24
@stdavis stdavis enabled auto-merge (squash) April 14, 2026 19:25
@stdavis stdavis disabled auto-merge April 14, 2026 19:30
@stdavis stdavis merged commit bb84227 into main Apr 14, 2026
6 checks passed
@stdavis stdavis deleted the copilot/add-plss-vector-tile-overlay branch April 14, 2026 19:30
stdavis pushed a commit that referenced this pull request Apr 14, 2026
Co-authored-by: stdavis <1326248+stdavis@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

Closes #253
@ugrc-release-bot ugrc-release-bot bot mentioned this pull request Apr 14, 2026
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.

Enhancement: add PLSS Vector Tile service as an Overlay option

2 participants