X-lab2017 / open-digger-website

Official Website for OpenDigger Community
https://open-digger.cn
Apache License 2.0
6 stars 6 forks source link

docs: added content for bus-factor + technical fork (EN + ZH) #25

Closed Neilblaze closed 1 month ago

Neilblaze commented 2 months ago

Description

Added the Bus Factor section into the metrics section. I've tried to preserve the quality of content by taking references from the existing ones (e.g. Technical Fork). Do note that the Chinese version is yet to be translated, so once this is approved, I'll push in the changes for CN/ZH variant.

UPDATE (04.07.24) → Added the Chinese (CN / ZH) translation of both Bus Factor & Technical Fork 🎉

🦄 Tested on local, build running successfully. PR is ready to be merged!

Ref : Slack Discussion / cc: @Peng99999 ○ Partially resolves #24

Before submitting the PR, please take the following into consideration

Neilblaze commented 2 months ago

/preview

github-actions[bot] commented 2 months ago

🪧The website for this PR is deployed at https://open-digger.cn/pull_25/

Neilblaze commented 2 months ago

🪧The website for this PR is deployed at https://open-digger.cn/pull_25/

@peng99999 the direct preview link is this from pull_25.

Peng99999 commented 2 months ago

@Neilblaze Well done! You can use translation software first (such as ChatGPT). If any modifications are needed in the Chinese documents, we will handle some of the language adjustments (polishing) ourselves afterwards.

Neilblaze commented 2 months ago

@Peng99999 ptal

Neilblaze commented 2 months ago

/preview

github-actions[bot] commented 2 months ago

🪧The website for this PR is deployed at https://open-digger.cn/pull_25/

frank-zsy commented 2 months ago

@Neilblaze I found that if we do not modify the base URL config in the config file, the redirections are broken in the preview website both yours and with 'preview` command. Is that right?

frank-zsy commented 2 months ago

/preview

github-actions[bot] commented 2 months ago

🪧The website for this PR is deployed at https://open-digger.cn/pull_25/

frank-zsy commented 2 months ago

/preview

github-actions[bot] commented 2 months ago

🪧The website for this PR is deployed at https://open-digger.cn/pull_25/

frank-zsy commented 2 months ago

@Neilblaze The workflow has been fixed. Right now, PR preview website will have a correct base URL to make sure all the relative links in the preview website are properly set. In order to make it take effect, I need to rebase and push back in this PR~

Neilblaze commented 2 months ago

@Neilblaze I found that if we do not modify the base URL config in the config file, the redirections are broken in the preview website both yours and with 'preview` command. Is that right?

@frank-zsy Yes, exactly! That's the root reason. I was planning to raise a PR, but I'm glad that you fixed it 😄

Neilblaze commented 2 months ago

@frank-zsy, @Peng99999 can we get this merged so that I can start working on adding new metrics :)

Peng99999 commented 2 months ago

@Neilblaze There are several places where the translation would be better as follows: "Bus factor" translated to "巴士因子"; "Technical Fork" translated to "技术分支".

frank-zsy commented 2 months ago

@Peng99999 Since @Neilblaze do not know Chinese, I think we should help with the Chinese part. I will work on this PR about the Chinese part then we can merge this PR.

Neilblaze commented 2 months ago

@frank-zsy That'd be extremely helpful, meanwhile as an update from my end, I've already started on this. Will ship a PR once done :)

frank-zsy commented 1 month ago

/preview

github-actions[bot] commented 1 month ago

🪧The website for this PR is deployed at https://open-digger.cn/pull_25/

frank-zsy commented 1 month ago

The PR now LGTM. Thanks for the docs. And I think ChatGPT is really good in translation~