Moving Giscus critical variables to .env #140
Music47ell
started this conversation in
Ideas
Replies: 1 comment 1 reply
-
Yes, let's do that. Noticed the unwanted comment as well 😆 |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
I think we should move Giscus critical variables to a
.env
file to prevent unwanted comments like this one.What I think happened is that someone cloned or forked this repository and tested the comment system without changing the settings inside https://github.com/timlrx/tailwind-nextjs-starter-blog/blob/master/data/siteMetadata.js causing their comment to appear in this template Discussion tab.
I propose that we do the following:
.env
file:repo, repositoryId, category, categoryId
.env.example
file similar to this one https://github.com/leerob/leerob.io/blob/main/.env.example and add the appropriate variables.This template is gaining a lot of attention and is increasing in popularity. Keeping the variables the way they are right now could cause more unwanted comments to appear and even spam.
I tested adding a
.env
file locally and it works great. I can send a PR soon if you agree with my proposal.Beta Was this translation helpful? Give feedback.
All reactions