Skip to content

French Rando : Add missing text + some tweaks#5327

Merged
leggettc18 merged 8 commits intoHarbourMasters:develop-blairfrom
PurpleHato:RandoFrenchFix
Apr 7, 2025
Merged

French Rando : Add missing text + some tweaks#5327
leggettc18 merged 8 commits intoHarbourMasters:develop-blairfrom
PurpleHato:RandoFrenchFix

Conversation

@PurpleHato
Copy link
Member

@PurpleHato PurpleHato commented Apr 5, 2025

Fixes #5309
And probably all other subsequent issues from missing texts

I think I don't miss anything? I hope so

Build Artifacts

PurpleHato and others added 6 commits April 5, 2025 12:48
…em.cpp

Co-authored-by: Extloga <141232749+Extloga@users.noreply.github.com>
Co-authored-by: Extloga <141232749+Extloga@users.noreply.github.com>
Co-authored-by: Extloga <141232749+Extloga@users.noreply.github.com>
@Tangeek42
Copy link

I found another missing text, I think. It seems related anyway ? In French the shops won't display the yellow text indicating the price. Out of curiosity I checked the German and it's even worse, all the text is bugged out, but I'm guessing it's because of something else.

fra
deu
eng

@PurpleHato
Copy link
Member Author

I found another missing text, I think. It seems related anyway ? In French the shops won't display the yellow text indicating the price. Out of curiosity I checked the German and it's even worse, all the text is bugged out, but I'm guessing it's because of something else.

The issue emerge from a refactor that has been done with Rando V3 and is not related to missing text, the fix for the price thing is there: #5312

@Tangeek42
Copy link

I found another missing text, I think. It seems related anyway ? In French the shops won't display the yellow text indicating the price. Out of curiosity I checked the German and it's even worse, all the text is bugged out, but I'm guessing it's because of something else.

The issue emerge from a refactor that has been done with Rando V3 and is not related to missing text, the fix for the price thing is there: #5312

My bad then ! Should've looked.

@leggettc18
Copy link
Contributor

I just merged #5312 and it looks like there's some conflicts, hopefully nothing too major but let me know if it's too hairy and I'll see what I can do to help.

@PurpleHato
Copy link
Member Author

I just merged #5312 and it looks like there's some conflicts, hopefully nothing too major but let me know if it's too hairy and I'll see what I can do to help.

Should be okay to merge now, I wouldn't mind a second eye to be extra safe tho

@leggettc18
Copy link
Contributor

Sounds good, soon as it builds I'll get right on it.

Copy link
Contributor

@Pepper0ni Pepper0ni left a comment

Choose a reason for hiding this comment

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

I can't speak for the contents of the translation, but it seems fine on a technical level.

@PurpleHato
Copy link
Member Author

Yup, I was worried for the technical side of it, since I resolved the conflict directly from the web interface
Just wanted to be safe on that
Thanks PepperOni o7

@leggettc18 leggettc18 merged commit 4acbf79 into HarbourMasters:develop-blair Apr 7, 2025
6 checks passed
@PurpleHato PurpleHato deleted the RandoFrenchFix branch April 7, 2025 22:40
Varuuna pushed a commit to Varuuna/Shipwright that referenced this pull request Apr 12, 2025
* Should cover everything, I hope

* One last

* final tweaks for real this time

* Update soh/soh/Enhancements/randomizer/3drando/hint_list/hint_list_item.cpp

Co-authored-by: Extloga <141232749+Extloga@users.noreply.github.com>

* Update soh/soh/Enhancements/randomizer/3drando/shops.cpp

Co-authored-by: Extloga <141232749+Extloga@users.noreply.github.com>

* Update soh/soh/Enhancements/randomizer/3drando/shops.cpp

Co-authored-by: Extloga <141232749+Extloga@users.noreply.github.com>

* TODO_TRANSLATE is a macro no a string Hato

---------

Co-authored-by: Extloga <141232749+Extloga@users.noreply.github.com>
krazyjakee pushed a commit to krazyjakee/OOT that referenced this pull request Sep 6, 2025
* Should cover everything, I hope

* One last

* final tweaks for real this time

* Update soh/soh/Enhancements/randomizer/3drando/hint_list/hint_list_item.cpp

Co-authored-by: Extloga <141232749+Extloga@users.noreply.github.com>

* Update soh/soh/Enhancements/randomizer/3drando/shops.cpp

Co-authored-by: Extloga <141232749+Extloga@users.noreply.github.com>

* Update soh/soh/Enhancements/randomizer/3drando/shops.cpp

Co-authored-by: Extloga <141232749+Extloga@users.noreply.github.com>

* TODO_TRANSLATE is a macro no a string Hato

---------

Co-authored-by: Extloga <141232749+Extloga@users.noreply.github.com>
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.

4 participants