Skip to content

Hide references to the Measure tool (MAP-8694)#117

Merged
michaelpnelson merged 1 commit intomasterfrom
hide_measure_tool
Feb 4, 2026
Merged

Hide references to the Measure tool (MAP-8694)#117
michaelpnelson merged 1 commit intomasterfrom
hide_measure_tool

Conversation

@michaelpnelson
Copy link
Contributor

The Measure tool was inadvertently broken by an update to the Leaflet library in PR bcgov/smk#188. This comments out some references to the Measure tool in SMK-CLI documentation to avoid confusion.

Copy link

Choose a reason for hiding this comment

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

broken link for Creating an SMK App

Copy link
Contributor Author

Choose a reason for hiding this comment

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

What looks broken? It looks fine to me - the link in the published page (https://bcgov.github.io/smk-cli/edit-an-app.html) loads the Creating page.

One thing I recently learned is that changes made to GitHub Pages files like this aren't immediately visible when looking at github.io Pages sites (ours for SMK-CLI is https://bcgov.github.io/smk-cli/). We need to publish the Pages site to see changes. It looks like publishing runs a build that converts this Markdown source into HTML and styles. This might be related to what you're seeing regarding broken links.

@michaelpnelson michaelpnelson merged commit fd9ef8c into master Feb 4, 2026
5 checks passed
@michaelpnelson michaelpnelson deleted the hide_measure_tool branch February 4, 2026 23:45
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.

4 participants