-
Notifications
You must be signed in to change notification settings - Fork 439
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
Update: the intro to storybook vue/chinese documentation #680
Conversation
@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 |
@jonniebigodes No problem.I will translate the rest of the intro of Vue part-time, and ping you as soon as it's finished. |
@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. |
@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. |
@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 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. |
The version has been updated. |
@jonniebigodes In case you don't see the information above. I @ you again. and hope to hear the message that can be merged. |
@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? |
@jonniebigodes That's ok. I'm glad to modify the required changes. |
There was a problem hiding this 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
#👇 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 }} |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
@jonniebigodes Hello, I wonder when it will be merged, or is there anything to be fixed? |
@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. |
@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 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! |
@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 |
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.