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

docs(open_graph): update options #1233

Closed
wants to merge 7 commits into from

Conversation

SukkaW
Copy link
Member

@SukkaW SukkaW commented Dec 1, 2019

Check List

Please read and check followings before submitting a PR.

  • I want to publish my theme on Hexo official website.
    • I have read the theme publishing doc.
    • name is unique.
    • link URL is correct.
    • preview URL is correct.
    • preview URL web site is rendered correctly.
    • Add a screenshot to source/themes/screenshots.
    • Screenshot filename is same as value of name.
    • Screenshot size is 800 * 500.
    • Screenshot file format is png.
  • I want to publish my plugin on Hexo official website.
  • Others (Update, fix, translation, etc...)

Update options supported in open_graph(). Will add to other languages after reviewed.

hexojs/hexo#3808 is also included.

@SukkaW SukkaW requested a review from curbengh December 1, 2019 09:59
@SukkaW SukkaW added this to the v4.1.0 milestone Dec 1, 2019
@SukkaW SukkaW mentioned this pull request Dec 1, 2019
1 task
source/docs/helpers.md Outdated Show resolved Hide resolved
source/docs/helpers.md Outdated Show resolved Hide resolved
@SukkaW SukkaW marked this pull request as ready for review December 2, 2019 12:42
@SukkaW SukkaW requested a review from curbengh December 2, 2019 12:42
@SukkaW
Copy link
Member Author

SukkaW commented Dec 7, 2019

@curbengh Rebased and update for all the languages. It is now ready to go.

@stevenjoezhang
Copy link
Member

Conflict with #1731

@SukkaW SukkaW closed this Jul 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants