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

Support sub-path URL configuration with popular posts feature #19

Merged
merged 1 commit into from Apr 18, 2019
Merged

Support sub-path URL configuration with popular posts feature #19

merged 1 commit into from Apr 18, 2019

Conversation

@lulzneko
Copy link
Contributor

@lulzneko lulzneko commented Apr 17, 2019

This pull request is to enhance the features of popular articles.
This enhances the behavior when the website is placed in the sub-path rather than at the top of the domain.

For example, when a website is placed on https://example.com/sub/20019/04/18/title, the feature of popular article works as follows and does not match.
-hexo-related-popular-posts: 2019/04/18/title
-ga-analytics: sub/2019/04/18/title

Use Hexo's URL configuration root to enhance popular article functionality. This configuration includes setting up the website subpath.

Hexo URL settings ("Sub-path" is written as "subdirectory" in the Hexo documentation)
https://hexo.io/docs/configuration#URL

Thank you for developing a wonderful plugin.
This plugin is an innovation in the static site generator world! !

@coveralls
Copy link

@coveralls coveralls commented Apr 17, 2019

Coverage Status

Coverage remained the same at 78.947% when pulling 5a16779 on lulzneko:master into d9e7687 on tea3:master.

@tea3 tea3 merged commit 2f33edc into tea3:master Apr 18, 2019
1 check passed
@tea3
Copy link
Owner

@tea3 tea3 commented Apr 18, 2019

@lulzneko
Pull Requestとご指摘を頂きましてありがとうございました!本日中にnpm versionのアップデートさせていただきますので宜しくお願いします^^ プラグインをご利用いただき光栄です!

@lulzneko
Copy link
Contributor Author

@lulzneko lulzneko commented Apr 19, 2019

@tea3 npm の バージョンアップありがとうございます。
動作確認することができました!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants