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 french vuejs translation #434

Conversation

jeanphi-baconnais
Copy link

No description provided.

@jeanphi-baconnais jeanphi-baconnais marked this pull request as ready for review January 11, 2021 08:08
@jeanphi-baconnais jeanphi-baconnais changed the title WIP - update french vuejs translation update french vuejs translation Jan 11, 2021
@jonniebigodes
Copy link
Collaborator

@jeanphibaconnais wow. I was checking and you updated this in a heartbeat. Really thankful for the the time you put into the pull request and how quickly you managed to get updated. A heads up for you and @Romainpetit i'm going to start updating the content based on #431 next week. So what this means is that @Romainpetit if you could leave the Get started section for last and for you @jeanphibaconnais i'll let you know once it's updated. I'll ping you when it's merged and we can start migrating the translations as well. Sounds good?

Stay safe both of you

@jeanphi-baconnais
Copy link
Author

@jeanphibaconnais wow. I was checking and you updated this in a heartbeat. Really thankful for the the time you put into the pull request and how quickly you managed to get updated. A heads up for you and @Romainpetit i'm going to start updating the content based on #431 next week. So what this means is that @Romainpetit if you could leave the Get started section for last and for you @jeanphibaconnais i'll let you know once it's updated. I'll ping you when it's merged and we can start migrating the translations as well. Sounds good?

Stay safe both of you

Yes it's ok for me

@jonniebigodes
Copy link
Collaborator

@jeanphibaconnais the pull request (#435) addresses the issue i've mentioned above. If you could adjust accordingly we could get this moving and get it updated and hopefully released soon. Sounds good?

Stay safe

@jeanphi-baconnais
Copy link
Author

Hi @jonniebigodes , i've just pushed modifications about your pull request #435 😄

@jonniebigodes
Copy link
Collaborator

@jeanphibaconnais with a recent change we've consolidated the documentation across Storybook and with that some changes were introduced. As you may have seen there's some conflicts in the pull request. When you get the chance could you please adjust accordingly?

Hoping to see this updated and merged.

Stay safe

@jonniebigodes
Copy link
Collaborator

@jeanphibaconnais in light of the previous comment above, a recent pull request (#456 ) was introduced to fine tune the documentation. When you're able if you could pull in the changes and adjust accordingly we would really appreciate it.

Let me know.

Stay safe

@jeanphi-baconnais
Copy link
Author

jeanphi-baconnais commented Mar 2, 2021

Hi @jonniebigodes i will try to work on this pull request this week ! I''m sorry, i changed work so february was a month with many things to finish 😁

@jonniebigodes
Copy link
Collaborator

@jeanphibaconnais no worries. Just let me know once you managed to get this across and we'll go from there. Wow, that seems like awesome news. Congrats on the new job 🎉.

@jeanphi-baconnais
Copy link
Author

Thanks 🙂 i'll tell you this week I hope

@jeanphi-baconnais
Copy link
Author

Hi @jonniebigodes i've just correct conflicts and take the PR 456 but actually there is a problem during build application.
3 files have this error : Can't resolve '../../../site-metadata'
File: src/components/composite/Footer.js
File: src/components/basics/LogoStorybook.js
File: src/components/screens/NotFoundScreen/NotFoundScreen.js
I verify that i have all the modifications ...

@jonniebigodes
Copy link
Collaborator

jonniebigodes commented Mar 5, 2021

@winkerVSbecks could you coordinate with @jeanphibaconnais and see what might be the issue here? Thanks in advance.

@winkerVSbecks
Copy link
Member

@jeanphibaconnais you need to run yarn extract-sb-docs-metadata

With the recent migration we now need to pull in some info from main storybook repo. I added instructions for this in the readme https://github.com/chromaui/learnstorybook.com#develop

@jeanphi-baconnais
Copy link
Author

Hi, thanks @winkerVSbecks , it's now ok 👍

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.

@jeanphibaconnais some items related to this and we'll be good to go:

  • If you could adjust the conclusion, more specifically the wording in this heading to something equivalent to "Who made the Intro to Storybook tutorial?"
  • Also I saw that you committed your yarn.lock, if you could remove it I would appreciate it as well.

Let me know once these are taken care of and I'll merge this.

Sounds good?

content/intro-to-storybook/vue/fr/composite-component.md Outdated Show resolved Hide resolved
@Dschungelabenteuer
Copy link
Contributor

@jeanphibaconnais Feel free to ping me anytime if you need help translating the updated version or some proofreading ✌️

@jeanphi-baconnais jeanphi-baconnais force-pushed the update-french-vuejs-translation- branch 2 times, most recently from 4320024 to 22c935c Compare May 6, 2021 05:48
@jeanphi-baconnais jeanphi-baconnais changed the title update french vuejs translation Draft - update french vuejs translation May 6, 2021
@jeanphi-baconnais
Copy link
Author

Hi, i start the french translation. I've just rebase and clean commit. I try to finish this one quickly.

@jeanphi-baconnais
Copy link
Author

Hi @Dschungelabenteuer, i've just finish a first version of the translation. I try to keep some english words, like 'story', 'addon' (after introducing the french word). If you have some time to read those modifications, it would be welcome 😄

@jonniebigodes
Copy link
Collaborator

@jeanphibaconnais sorry for the "radio silence" but I've been busy with other items of documentation. I saw that the feedback is being addressed which is really awesome. Just one small item that needs to be addressed. If you could factor in #504 it would help us out greatly. Let me know once the change is introduced and we'll go from there.

@jeanphi-baconnais
Copy link
Author

Hi @jonniebigodes , no problem :)
I've just changed the repo for this PR504.

@jeanphi-baconnais jeanphi-baconnais changed the title Draft - update french vuejs translation update french vuejs translation Jul 13, 2021
@jeanphi-baconnais
Copy link
Author

Hi @jonniebigodes , have you see my comments on this PR ?

fix : change snippet in english
@jeanphi-baconnais jeanphi-baconnais force-pushed the update-french-vuejs-translation- branch from 0eb3b20 to 9c71cad Compare December 6, 2021 11:55
@jonniebigodes
Copy link
Collaborator

Closing this pull request as mentioned in the issue @jeanphibaconnais! Looking forward to the updated translation.

Have a great weekend!

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.

4 participants