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.

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

Remove recommendation to add reCaptcha to staticman.yml #1843

Closed
wants to merge 1 commit into from

Conversation

danamlewis
Copy link

Suggestion for docs - adding to staticman causes reCaptcha to not show in the comment form; only _config.yml needed modifying.

(Adding to staticman blocks the reCaptcha from showing in the site - or at least did in my implementation of the template. Otherwise following these docs worked great for adding reCaptcha in.)

And - huge thank you for this site template and the fantastic documentation! I've highlighted it here as one of the reason's we chose it for our project's blog.

(Adding to staticman blocks the reCaptcha from showing in the site)
@mmistakes
Copy link
Owner

Strange. As per Staticman's docs they're needed in its staticman.yml config.

https://staticman.net/docs/configuration

@danamlewis
Copy link
Author

Ok. It's possible that I implemented weirdly, somehow! But, reCaptcha is working successfully in mine without being in staticman, just _config.

@mmistakes
Copy link
Owner

Looks like you've hardcoded the reCaptcha values directly into the comments include. So it's not coming from _config.yml at all. Guessing there's more of that since your repo appears to be modified in a few places.

I'd be curious to hear what other's experiences are with configuring Staticman v2 with reCaptcha and if it deviates from what's in the docs.

@stale
Copy link

stale bot commented Oct 20, 2018

This issue has been automatically marked as stale because it has not had recent activity.

If this is a bug and you can still reproduce this error on the master branch, please reply with any additional information you have about it in order to keep the issue open.

If this is a feature request, please consider whether it can be accomplished in another way. If it cannot, please elaborate on why it is core to this project and why you feel more than 80% of users would find this beneficial.

This issue will automatically be closed in 7 days if no further activity occurs. Thank you for all your contributions.

@stale stale bot added the Status: Stale label Oct 20, 2018
@stale stale bot closed this Oct 27, 2018
okitem pushed a commit to okmalls/okmalls.github.io that referenced this pull request Sep 21, 2024
okitem pushed a commit to okmalls/okmalls.github.io that referenced this pull request Sep 21, 2024
## [7.1.0](cotes2020/jekyll-theme-chirpy@v7.0.1...v7.1.0) (2024-08-27)

### Features

* add Bluesky social links ([mmistakes#1759](cotes2020/jekyll-theme-chirpy#1759)) ([0102aba](cotes2020/jekyll-theme-chirpy@0102aba))
* add Reddit social option ([mmistakes#1836](cotes2020/jekyll-theme-chirpy#1836)) ([8673e13](cotes2020/jekyll-theme-chirpy@8673e13))
* add Threads social links ([mmistakes#1837](cotes2020/jekyll-theme-chirpy#1837)) ([e3a78b6](cotes2020/jekyll-theme-chirpy@e3a78b6))
* **analytics:** add fathom analytics ([mmistakes#1913](cotes2020/jekyll-theme-chirpy#1913)) ([befc4ce](cotes2020/jekyll-theme-chirpy@befc4ce))
* **dev:** add vscode tasks ([mmistakes#1843](cotes2020/jekyll-theme-chirpy#1843)) ([e4db1a1](cotes2020/jekyll-theme-chirpy@e4db1a1))
* **dev:** support vscode dev-container ([mmistakes#1781](cotes2020/jekyll-theme-chirpy#1781)) ([1e3d4a6](cotes2020/jekyll-theme-chirpy@1e3d4a6))
* **ui:** improve block quote layout ([80bd792](cotes2020/jekyll-theme-chirpy@80bd792))
* **ui:** improve visibility of inline code ([mmistakes#1831](cotes2020/jekyll-theme-chirpy#1831)) ([c876731](cotes2020/jekyll-theme-chirpy@c876731))
* **ui:** make `info-prompt` icon looks like the letter "i" ([mmistakes#1835](cotes2020/jekyll-theme-chirpy#1835)) ([a07a57e](cotes2020/jekyll-theme-chirpy@a07a57e))
* **ui:** set `<kbd>` font to 'Lato' ([64c7262](cotes2020/jekyll-theme-chirpy@64c7262))

### Bug Fixes

* adapt the giscus localization parameter ([mmistakes#1810](cotes2020/jekyll-theme-chirpy#1810)) ([0709854](cotes2020/jekyll-theme-chirpy@0709854))
* avoid caching pageviews data ([mmistakes#1849](cotes2020/jekyll-theme-chirpy#1849)) ([979f86c](cotes2020/jekyll-theme-chirpy@979f86c))
* remove extra dual-mode images from lightbox ([mmistakes#1883](cotes2020/jekyll-theme-chirpy#1883)) ([5c5910f](cotes2020/jekyll-theme-chirpy@5c5910f))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants