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(helper): add full_url_for #1092

Merged
merged 3 commits into from
Oct 16, 2019
Merged

Conversation

SukkaW
Copy link
Member

@SukkaW SukkaW commented Sep 18, 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.
    • 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...)

hexojs/hexo-util#84
hexojs/hexo#3701

Add usage & example for newly added full_url_for() helper.

Should be merged after hexo 4 is released.

@SukkaW SukkaW added the documentation Improvements or additions to documentation label Sep 18, 2019
@SukkaW SukkaW added this to the v4.0.0 milestone Sep 18, 2019
@SukkaW SukkaW requested review from curbengh and tomap September 18, 2019 07:45
curbengh
curbengh previously approved these changes Sep 18, 2019
source/docs/helpers.md Outdated Show resolved Hide resolved
@SukkaW
Copy link
Member Author

SukkaW commented Sep 18, 2019

Also, the zh-CN translations has been added in this PR.

@SukkaW SukkaW merged commit 8448996 into hexojs:master Oct 16, 2019
@SukkaW SukkaW deleted the full_url_for-docs branch October 29, 2019 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants