Skip to content

fix(share_menu): url issue of sharing links#367

Merged
neoFelhz merged 2 commits intoiblh:canaryfrom
yunjianyang:canary
May 26, 2017
Merged

fix(share_menu): url issue of sharing links#367
neoFelhz merged 2 commits intoiblh:canaryfrom
yunjianyang:canary

Conversation

@yunjianyang
Copy link
Contributor

@yunjianyang yunjianyang commented May 26, 2017

What kind of change does this PR introduce? (check one with "x")

  • Bug fix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

Does this PR introduce a breaking change? (check one with "x")

  • Yes
  • No

Description

url_for(path) will include the root automatically. So I delete the config.root before url_for(path).

Please refer to https://hexo.io/docs/helpers.html


Verification steps

No verification steps.

@yunjianyang yunjianyang changed the title Canary Fix problem when sharing to social network services May 26, 2017
@neoFelhz neoFelhz requested review from cubesky, iblh and neoFelhz May 26, 2017 03:33
@neoFelhz neoFelhz changed the title Fix problem when sharing to social network services fix(share_menu): url issue of sharing links May 26, 2017
@neoFelhz neoFelhz merged commit ad35ea6 into iblh:canary May 26, 2017
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.

2 participants