Skip to content

fix twitter shortcode #89

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
Closed
wants to merge 1 commit into from
Closed

fix twitter shortcode #89

wants to merge 1 commit into from

Conversation

@deining
Copy link
Contributor

@deining deining commented Feb 20, 2024

As of hugo v0.123.0, the basic example site fails with an error due to improper use of twitter shortcode.

& hugo Start building sites … hugo v0.123.0-3c8a4713908e48e6523f058ca126710397aa4ed5+extended linux/amd64 BuildDate=2024-02-19T16:32:38Z VendorInfo=gohugoio (...) ERROR The "twitter_simple" shortcode requires two named parameters: user and id. See "/home/andreas/hugoBasicExample/content/post/rich-content.md:26:1" Total in 59 ms Error: error building site: logged 1 error(s)

This PR fixes this error.
@deining
Copy link
Contributor Author

deining commented Feb 20, 2024

Just saw that this is a duplicate of #82, sorry.
Nevertheless, we should either address this issue or archive this repo, as suggested in #83.
@deining deining mentioned this pull request Feb 20, 2024
Closed
@coliff coliff closed this Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@deining @coliff