vuepress / vuepress-community

:bulb: Community supported ecosystem for VuePress
https://vuepress-community.netlify.app
MIT License
81 stars 62 forks source link

Fix #34: Redirect to lang if dialect is not found #42

Closed TiEul closed 4 years ago

TiEul commented 4 years ago

Summary

This change means that users with browsers that send an unknow region of a known language will still be forwarded to the correct overall language. For example, users with a browser that sends de-AT as a locale will still be forwarded to the de-DE locale, as the plugin will only take the lowercase de into account while ignoring the region-specific uppercase DE and AT.

Which package does this PR involve? (check one)

What kind of change does this PR introduce? (check at least one)

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

If yes, please describe the impact and migration path for existing applications:

The PR fulfills these requirements:

Other information:

https://github.com/vuepress/vuepress-community/issues/34

stale[bot] commented 4 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

TiEul commented 4 years ago

I can't wait any longer with this because it's fundamentally broken so I will just continue on a fork of vuepress-community myself. I am still happy to make a new pull request, please see the conversation in #34.

meteorlxy commented 4 years ago

@TiEul

Sorry I don't have so much time for open source projects these days.

Feel free to create your own fork and thanks for your contribution~

TiEul commented 4 years ago

@meteorlxy I totally understand that personal priorities shift but is there no one else from the Vue(Press) community who could potentially take over the maintenance of this package? I would consider it pretty important for VuePress and clearly there are still some issues with it that make it "less than ideal" for production.

I am a bit frustrated because I never even got a response about what was wrong with my commit which is also why I never followed-up with the issue in #44 because that would take a lot of time to prepare even though it's reliably reproducible. Again, I consider this commit more of a bug fix and I even have Google Analytics data to back up this lack of redirects for many browsers.

Anyway, my commit is still here now on a cleaned refork:

https://github.com/TiEul/vuepress-community/commit/ca5f7e3f085d96d26c42022b56957238629d5f31

meteorlxy commented 4 years ago

In fact, this org is NOT officially maintained by VuePress team (even though I'm a member of the them).

And the new VitePress just made me not so active on VuePress.

In addition, you must know that most open source projects can not be guaranteed, because they don't have any sponsors and cannot help their authors to make a live. Just for the open source spirit!:smile: