Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Plausible satellite integration: Add plausible script to the Satellite UI #6820

Closed
mobyvb opened this issue Feb 29, 2024 · 3 comments
Closed

Plausible satellite integration: Add plausible script to the Satellite UI #6820

mobyvb opened this issue Feb 29, 2024 · 3 comments
Assignees
@wilfred-asomanii
Labels
Plausible Satellite Integration

Comments

@mobyvb
Copy link
Member

mobyvb commented Feb 29, 2024

Blocked by #6819

Insert the Plausible "manual" script into the UI if analytics is enabled.

AC:

  • When analytics is enabled, execute Plausible code into the head tag of the Satellite UI app
    • example script tag for local development: <script defer data-domain="testing.storj.io" src="https://plausible.io/js/script.manual.local.js"></script>
    • based on the result of Plausible satellite integration: Review plausible code #6819, this script may be either hosted on plausible.io, or self-hosted on the satellite
    • it should be easy to configure the script for local vs. prod (in prod we would use script.manual.js but in local development we would use script.manual.local.js - may require adding some satellite configuration
@mobyvb mobyvb mentioned this issue Feb 29, 2024
Closed
@mobyvb mobyvb added the Needs Estimation Issue still needs story pointing label Feb 29, 2024
@wilfred-asomanii wilfred-asomanii self-assigned this Mar 7, 2024
@wilfred-asomanii wilfred-asomanii removed the Needs Estimation Issue still needs story pointing label Mar 7, 2024
@storj-gerrit
Copy link

storj-gerrit bot commented Mar 8, 2024

Change satellite/{console,analytics}: add plausible configs mentions this issue.

@storj-gerrit
Copy link

storj-gerrit bot commented Mar 8, 2024

Change web/satellite: add plausible script to Satellite UI mentions this issue.

storjBuildBot pushed a commit that referenced this issue Mar 11, 2024
@wilfred-asomanii
satellite/{console,analytics}: add plausible configs
cd7bb2c 
This change adds configs for plausible analytics and sends them to the
satellite UI.

Issue: #6820

Change-Id: Iae1fb10dfe8f939a2b9134f6fc879aa16e4ec6f0
storjBuildBot pushed a commit that referenced this issue Mar 15, 2024
@wilfred-asomanii
web/satellite: add plausible script to Satellite UI
5fbd466 
This change adds the plausible script to the web UI if analytics is
enabled.

Issue: #6820

Change-Id: I3c749a627f3efad7181d7c719d7a28fbe38f2dcb
@wilfred-asomanii
Copy link
Contributor

https://github.com/storj/storj/releases/tag/v1.101.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wilfred-asomanii wilfred-asomanii

Labels
Plausible Satellite Integration
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mobyvb @wilfred-asomanii