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

Update: the intro to storybook vue/chinese documentation #680

Merged
merged 12 commits into from
Jan 31, 2023

Conversation

fengnzl
Copy link
Contributor

@fengnzl fengnzl commented Dec 25, 2022

It will be my first time contributing to a project so I hope I can be useful! I updated vue/chinese get-started and simple-component documentations.

@jonniebigodes
Copy link
Collaborator

@fengnzl, this is fantastic. Thank you so much for the time and effort you've put into this pull request and for helping us update the translation 🙏 ! With that, I wanted to ask if you're also okay with updating the remaining tutorial sections. As you already started the work in this branch, feel free to use it to make the rest of the changes. And also, once it's done, ping me so that I can check with our community for a proofreader to take a look at this and get it merged and published. Does that work for you?

Looking forward to hearing from you.

Hope you have a great day.

Stay safe

@fengnzl
Copy link
Contributor Author

fengnzl commented Dec 29, 2022

@jonniebigodes No problem.I will translate the rest of the intro of Vue part-time, and ping you as soon as it's finished.

@jonniebigodes
Copy link
Collaborator

@fengnzl that's great to hear, I've been checking on and off the progress, and it's looking good so far. I'll take a look once you've finished, and we'll go from there.

@fengnzl
Copy link
Contributor Author

fengnzl commented Jan 5, 2023

@jonniebigodes I've finished translating the intro of Vue. Please take a look and check this. But It seems something went wrong in CI, I checked the error message and it maybe not caused by me. If there is something that needs to be fixed, I would handle that.

@jonniebigodes
Copy link
Collaborator

@fengnzl, thanks for taking care of the translation in such a short amount of time 🙏 ! I'm going to take a pass at it shortly. If you don't mind taking care of it, one thing left to update is to change the value here from 6.3 to 6.5. So that once the translation is merged, it doesn't incorrectly mention that is outdated.

Regarding the CI workflow, don't worry about it. I'm checking it to see what may be happening on our end. Thanks for letting me know, nonetheless.

@fengnzl
Copy link
Contributor Author

fengnzl commented Jan 8, 2023

The version has been updated.

@fengnzl
Copy link
Contributor Author

fengnzl commented Jan 12, 2023

@jonniebigodes In case you don't see the information above. I @ you again. and hope to hear the message that can be merged.

@jonniebigodes
Copy link
Collaborator

@fengnzl I saw the update. Sorry for not following up with you on this. I will reach out to the community later today and see if we can find a proofreader. Tomorrow morning I'll take a look at the translation in more detail and follow up with you if any changes are required on my end. Sounds good?

@fengnzl
Copy link
Contributor Author

fengnzl commented Jan 13, 2023

@jonniebigodes That's ok. I'm glad to modify the required changes.

Copy link
Collaborator

@jonniebigodes jonniebigodes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fengnzl from me, just one small item and this should be good to go. I sent out a callout to our community to see if someone was interested in proofreading it. To avoid having this pr being blocked from merging, I'm going to put a timer on it, so to say, if no one comes forward and follows up with you and me and proofreads it by end of next week. I'll merge it, and we'll polish it up with follow-up pull requests. Does that work for you?

Looking forward to hearing from you.

Hope you have a great week.

Stay safe

Comment on lines 103 to 105
#👇 Chromatic projectToken, see https://storybook.js.org/tutorials/intro-to-storybook/vue/en/deploy/ to obtain it
projectToken: ${{ secrets.CHROMATIC_PROJECT_TOKEN }}
token: ${{ secrets.GITHUB_TOKEN }}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fengnzl, one item here. If you could revert this link to where it was pointing, in other words to https://storybook.js.org/tutorials/intro-to-storybook/vue/zh-cn/deploy/ I'd appreciate it. The reasoning behind this is the following. We'd like for our readers to jump back to their native language if they want to learn more about how to set up Chromatic instead of having them jump to an English version, which may not be their native language. Sounds reasonable?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, I will fix it later and check other links.

@fengnzl
Copy link
Contributor Author

fengnzl commented Jan 31, 2023

@jonniebigodes Hello, I wonder when it will be merged, or is there anything to be fixed?

@jonniebigodes
Copy link
Collaborator

@fengnzl , I haven't forgotten about you, don't worry about it. I'll take a last pass at it today, merge it if nothing is required and follow up with you on it, as no one came forward to proofread it. Don't want to get you blocked on this as you already have it pretty much wrapped up.

@jonniebigodes
Copy link
Collaborator

@fengnzl I just took a final pass at your incredible work, and it looks good on my end. A couple of things I would like your feedback on. First, can you jump into our Discord Server and message me (same username) for a small item related to this? Second, I'm going to update the call to translations issue we have and reference you as a translator/contributor so that when future pull requests addressing your native language surface, we can sync up with you and help us out in proofreading it. Does that work for you?

Looking forward to hearing from you. I hope you have a fantastic day, and extremely thankful for this fantastic contribution.

Stay safe

@jonniebigodes jonniebigodes merged commit 59cd563 into chromaui:master Jan 31, 2023
@fengnzl
Copy link
Contributor Author

fengnzl commented Feb 1, 2023

@jonniebigodes Glad to hear that! I tried to join Discord but for some network reason I verify my email so many times and my account was locked, I reported it to the official, but I don't know when my account available. And I am glad to participate in follow-up translation projects related to Vue and proofread them.

Have a nice day!

@jonniebigodes
Copy link
Collaborator

@fengnzl, thanks for following up with me, and that's great to hear. We appreciate your willingness to help us proofread any follow-up pull requests. And don't worry about the Discord item for now. We have an alternative. I will reach out to you via the email you have listed in your GitHub profile, and we'll go from there. Sounds good? Be on the lookout for it.

I hope you have a great day.

Stay safe

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

Successfully merging this pull request may close these issues.

2 participants