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

Italian translations for vue of intro-to-storybook #756

Merged
merged 16 commits into from May 14, 2024

Conversation

Archetipo95
Copy link
Contributor

Huge thanks to @ludovicobesana for the React translations which I took for this Vue translation

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.

@Archetipo95, thank you so much for taking the time to put together this pull request and helping our community get a better understanding of Storybook by extending our translations. We truly appreciate it 🙏 !

I left a couple of small items for you to look into when you have a moment.

Also, @ludovicobesana, when you have a moment, can you take a pass at this pull request from a language standpoint so we can polish it and release it?

Hope both of you have a great day.

Stay safe.

content/intro-to-storybook/vue/it/get-started.md Outdated Show resolved Hide resolved
content/intro-to-storybook/vue/it/screen.md Outdated Show resolved Hide resolved
@Archetipo95
Copy link
Contributor Author

@jonniebigodes I fixed the issues that you mentioned in the comments. If you have something else feel free to give me a feedback!

@jonniebigodes
Copy link
Collaborator

@Archetipo95 appreciate you addressing the feedback so promptly 🙏 ! From my end, all is good. I'll message @ludovicobesana tomorrow morning my time and see if he has time to take a look into this, so that we're able to get the feedback addressed as soon as possible and publish the translation.

If he's unable to review it in a reasonable timeframe. I'll go ahead and merge this, and we can make some adjustments in a follow-up pull request.

Sounds reasonable to you?

Have a fantastic day.

Stay safe

@Archetipo95
Copy link
Contributor Author

Archetipo95 commented Apr 3, 2024

Absolutely, it sounds awesome! Addressing feedback promptly is crucial for maintaining momentum 🚀

Wishing you a fantastic day as well!

@ludovicobesana
Copy link
Contributor

@jonniebigodes I'll do the review by Sunday 7th.
Thanks again for your contribution @Archetipo95 🙏🏻

@jonniebigodes
Copy link
Collaborator

Appreciate it @ludovicobesana 🙏

@ludovicobesana
Copy link
Contributor

@Archetipo95 I'm looking it right now in code from here: https://github.com/Archetipo95/learnstorybook.com/tree/master

@ludovicobesana
Copy link
Contributor

ludovicobesana commented Apr 7, 2024

@Archetipo95 I've done a quick review, could you please check the feedbacks that I've added?
Thank you! 🙏🏻

Now I need to go but tomorrow I'll check if I found other things that can be improved.

cc. @jonniebigodes

@Archetipo95
Copy link
Contributor Author

@jonniebigodes I resolved all the suggestions that @ludovicobesana raised. If you want to close it feel free :D

@ludovicobesana
Copy link
Contributor

@jonniebigodes I resolved all the suggestions that @ludovicobesana raised. If you want to close it feel free :D

Hi @Archetipo95, I see only one commit with one fix: e6711b4

Can you please look a the other feedbacks that I've added for you?
Thank you! 🙏🏻

@Archetipo95
Copy link
Contributor Author

Hi :D @ludovicobesana I did both fixes in one commit
Am I missing something?
Screenshot 2024-04-15 at 22 05 10

Copy link
Contributor

@ludovicobesana ludovicobesana left a comment

Choose a reason for hiding this comment

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

content/intro-to-storybook/vue/it/simple-component.md Outdated Show resolved Hide resolved
content/intro-to-storybook/vue/it/simple-component.md Outdated Show resolved Hide resolved
content/intro-to-storybook/vue/it/simple-component.md Outdated Show resolved Hide resolved
content/intro-to-storybook/vue/it/simple-component.md Outdated Show resolved Hide resolved
content/intro-to-storybook/vue/it/simple-component.md Outdated Show resolved Hide resolved
content/intro-to-storybook/vue/it/simple-component.md Outdated Show resolved Hide resolved
content/intro-to-storybook/vue/it/simple-component.md Outdated Show resolved Hide resolved
@ludovicobesana
Copy link
Contributor

Hi :D @ludovicobesana I did both fixes in one commit Am I missing something? Screenshot 2024-04-15 at 22 05 10

@Archetipo95 I've re-added this points and also if you can check also the other new parts of translations that could be great!

@Archetipo95
Copy link
Contributor Author

Done! Sorry I messed something and I was not seeing your suggestions :D thanks for tagging me @ludovicobesana
Now everything should be resolved, I accepted all of your suggestions, thanks for taking time to check this PR

@ludovicobesana
Copy link
Contributor

Done! Sorry I messed something and I was not seeing your suggestions :D thanks for tagging me @ludovicobesana Now everything should be resolved, I accepted all of your suggestions, thanks for taking time to check this PR

Cool thank you, did you check the other documentation files too?
Just to have a double check.

@jonniebigodes
Copy link
Collaborator

@Archetipo95 and @ludovicobesana can you give me a status update on this pull request so that we can continue to work on it or get it out?

Let me know and we'll go from there.

Have a fantastic day the both of you.

@Archetipo95
Copy link
Contributor Author

@jonniebigodes I did the suggestions that @ludovicobesana made and I'm done with this Pull Request for now. :)

@ludovicobesana
Copy link
Contributor

Hi @jonniebigodes if @Archetipo95 has resolved all the points and reviewed the other files I think that we can merge it 🚀

@jonniebigodes
Copy link
Collaborator

@Archetipo95, I appreciate you following up with me on this, and I will merge this so we can get it published.

One small thing to take care of, @Archetipo95. If you're ok with it, can you message me directly (same username) on our Discord Server so that I can follow up with you on this pull request?

@ludovicobesana cannot thank you enough for your time in getting this proofread 🙇

Hope both of you have a great day.

Stay safe

@jonniebigodes jonniebigodes merged commit 4577b05 into chromaui:master May 14, 2024
1 check failed
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.

None yet

3 participants