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

Create fr_fr.json #106

Open
wants to merge 2 commits into
base: 1.17
Choose a base branch
from
Open

Conversation

Water-Kitten
Copy link

French translation for Sandwichable on 1.17

As I did for the previous version, I made a new french translation for the 1.17 version.

I hope it'll work great, I never did any "pull request" before and there's not a lot of explications/tutorials in french about it. Sorry if I did it wrong ! Just tell me and I'll correct it right.

Hope it will help you !

French translation for Sandwichable on 1.17
Comment on lines 270 to 283

© 2021 GitHub, Inc.
Terms
Privacy
Security
Status
Docs

Contact GitHub
Pricing
API
Training
Blog
About

Choose a reason for hiding this comment

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

This appears to be a mistake where too much was copied when copying a template for the lang file. As it is the footer of GitHub.

Suggested change
© 2021 GitHub, Inc.
Terms
Privacy
Security
Status
Docs
Contact GitHub
Pricing
API
Training
Blog
About

Copy link
Author

Choose a reason for hiding this comment

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

So I need to remove this footer and it will be okay ? (Sorry, real noob here...)

Choose a reason for hiding this comment

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

Yes, at least the file. I have not verified the actual translations since I don't speak french. Use en_us.json for reference. The closing curly bracket should be the last line of the file.

@Water-Kitten
Copy link
Author

Like this? I've use the en_us.json for the first one too, I don't where it found those last line.

I remove them manualy on the file. Would it have done the same thing if I had clicked on "Add suggestion to batch" or it is for something else?

Thank you for your help and your correction !

@EcoBuilder13
Copy link

Yes, that fixes it. The "Add suggestion to batch" would of done the same thing for the most part. It's used so you can put all of the requested changes you accept into the same commit automatically and it adds the people who suggested the change as co-authors. I think you got those lines because you copied the file from a browser tab and it highlighted the bottom row of the webpage so it got copied.

@Water-Kitten
Copy link
Author

Ow sorry, your name will not be add as co-author 'cause I changed it manualy...

Thanks again for your help and explanations, realy appreciate it!

@FoundationGames
Copy link
Owner

Hello, the 1.17 branch is not actually updated to 1.17 - it is equivalent to the indev branch.
Therefore when I update the mod to 1.17, this translation might need to be updated. Sorry for the misleading branch titles, I hope I can hear back. There is no ETA on when 1.17 will be completed, but it has not started thus far.

@Water-Kitten
Copy link
Author

Okay, I'll try to pass this one on "indev" so you'll can have the current translation up-to-date anyway !
Don't hesitate to notice me when any updates or changes are needed !

Sorry to have bothered the pulls with all my translations' pull request...

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

Successfully merging this pull request may close these issues.

3 participants