Skip to content

Conversation

@DerekNonGeneric
Copy link
Member

this is a wip of adding some analytics (goatcounter), but still left to do would be locally hosting the count.js script

@DerekNonGeneric DerekNonGeneric added 🟨 Code: JS/TS PRs or issues regarding JavaScript/TypeScript code 📖 Category: Documentation Improvements or additions to documentation 🚧 Status: Work In Progress (WIP) undefined ✨ Type: Feature undefined labels Nov 24, 2024
@netlify
Copy link

netlify bot commented Nov 24, 2024

Deploy Preview for gh-pages-openinf ready!

Name Link
🔨 Latest commit 8c1caf4
🔍 Latest deploy log https://app.netlify.com/sites/gh-pages-openinf/deploys/6746c611cb8a5b00087a2c94
😎 Deploy Preview https://deploy-preview-1511--gh-pages-openinf.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@DerekNonGeneric
Copy link
Member Author

DerekNonGeneric commented Nov 27, 2024

we need to add the assets/js/vendor/ dir to the deepsource exclude patter here:

🔗 https://docs.deepsource.com/docs/configuration#exclude_patterns

Signed-off-by: OpenINF-bot <[email protected]>
@OpenINFbot OpenINFbot merged commit da43014 into OpenINF:live Nov 27, 2024
10 checks passed
@DerekNonGeneric DerekNonGeneric deleted the feat/add-goatcounter branch November 27, 2024 07:18
DerekNonGeneric added a commit to DerekNonGeneric/open.inf.is that referenced this pull request Nov 29, 2024
PR URL: OpenINF#1511
Reviewed-by: OpenINF-bot <[email protected]>

---------

Signed-off-by: OpenINF-bot <[email protected]>
Co-authored-by: OpenINF-bot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📖 Category: Documentation Improvements or additions to documentation 🟨 Code: JS/TS PRs or issues regarding JavaScript/TypeScript code 🚧 Status: Work In Progress (WIP) undefined ✨ Type: Feature undefined

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants