Skip to content

chore: add plausible analytics for stac-map#355

Merged
gadomski merged 3 commits intomainfrom
chore/plausible-stac-map
Mar 5, 2026
Merged

chore: add plausible analytics for stac-map#355
gadomski merged 3 commits intomainfrom
chore/plausible-stac-map

Conversation

@aboydnw
Copy link
Member

@aboydnw aboydnw commented Mar 4, 2026

Agh! @gadomski Sorry for all the back and forth. I moved too quickly through all the pages. For stac-map, instead of removing the script today I should have just changed the domain. I added "stac-map" as a site in Plausible and changed the configuration here to match.

Checklist

  • Code is formatted (yarn format)
  • Code is linted (yarn lint)
  • Code builds (yarn build)
  • Tests pass (yarn test)
  • Commit messages and/or this PR's title are formatted per Conventional Commits

Add the standard Plausible script include and configure tracking with the stac-map site identifier.
@aboydnw aboydnw requested a review from gadomski as a code owner March 4, 2026 21:01
@github-actions github-actions bot added the chore label Mar 4, 2026
@gadomski
Copy link
Collaborator

gadomski commented Mar 4, 2026

I just tried your link and got this error:

image

Replace the legacy Plausible tag with the current stac-map site-specific snippet.
@aboydnw
Copy link
Member Author

aboydnw commented Mar 5, 2026

That's just the automated testing functionality. But to make it easy, I changed the URL in Plausible to developmentseed.org/stac-map so it should be able to automatically verify installation once this is merged.

But Plausible also recently updated its script so I updated this PR to include the new script.

@gadomski gadomski enabled auto-merge (squash) March 5, 2026 23:36
@gadomski gadomski merged commit cec4535 into main Mar 5, 2026
4 checks passed
@gadomski gadomski deleted the chore/plausible-stac-map branch March 5, 2026 23:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants