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

Re:Re:Fix & Add Japanese Game Titles. #12089

Merged
merged 1 commit into from
Dec 15, 2024

Conversation

nassau-tk
Copy link
Contributor

Description of Changes

Fixed a large number of Japanese game titles.
And, Re-fix [ name-en: ].
*Trial & Demo will fix later.

Rationale behind Changes

This is a difficult task for non-Japanese people, so I did it myself.
*I'm JP.

Suggested Testing Steps

Put on the "resources" folder and check booting PCSX2.
If the .yaml file has problem then PCSX2 will not boot(Crash).

This is [Third attempt] lol

@TellowKrinkle
Copy link
Member

TellowKrinkle commented Dec 15, 2024

You may want to run ajv locally to fix the issues instead of waiting for the CI to run after each one

npm install ajv-cli
ajv validate -s ./pcsx2/Docs/gamedb-schema.json --spec=draft2020 -d ./bin/resources/GameIndex.yaml

Edit: I went through and fixed the validation issues

@TellowKrinkle
Copy link
Member

Also, you can just add more commits instead of opening a new PR for each one
We can squish them all into one commit when we merge

@nassau-tk
Copy link
Contributor Author

Oh!
Thank you for your fixing!!
I'm glad for your kindness.
And, I'm so sorry for my poor knowlddge.

I will more learn about github!

Thank you.

@TellowKrinkle
Copy link
Member

I made a few more changes:

  • It appears you did a find and replace from ! to (fullwidth), and it affected both English and Japanese titles. I've undone the change for all non-Japanese titles
    • I did not undo this for "Strawberry Panic! [初回限定版]" and "Strawberry Panic! [通常版]", since they are Japanese games, even though they use English for the entire title. Should I?
  • It appears you accidentally undid the last few commits that changed the gamedb. I've re-applied those.

Let me know if those make sense to you and if so, I'll squish them all into the main commit so the change list is a bit smaller.

@nassau-tk
Copy link
Contributor Author

I assume that name: and name-sort: in Japanese games are all full-width "!".

I think it's okay to use half-width characters for name-en: in Japanese games as well.

@TellowKrinkle
Copy link
Member

I assume that name: and name-sort: in Japanese games are all full-width "!".

Correct.

@TellowKrinkle
Copy link
Member

I've squished all the commits into one.
If that looks good to you, let me know and I'll merge it.

@nassau-tk
Copy link
Contributor Author

Yes, It seems very nice.

The most important purpose this time is to display Japanese and sort based on Japanese as much as possible.

It's was done by your help!

In the future, I would like to modify the Demo/Trial version and make minor corrections.

@TellowKrinkle TellowKrinkle merged commit 3cafd2d into PCSX2:master Dec 15, 2024
13 checks passed
@TellowKrinkle
Copy link
Member

In the future, I would like to modify the Demo/Trial version and make minor corrections.

Sounds good. Make a new PR with that when you're ready.

@pgert
Copy link
Contributor

pgert commented Dec 15, 2024

@nassau-tk:
You might want to check this out, if you haven't done so already:
https://forums.pcsx2.net/Thread-GameDB-Gamefixes-update-discussion?pid=566963#pid566963

@nassau-tk
Copy link
Contributor Author

@pgert

Shouldn't I have fixed it personally?

@pgert
Copy link
Contributor

pgert commented Dec 20, 2024

@pgert

Shouldn't I have fixed it personally?

I ment if there is something there that is new to you.

@nassau-tk
Copy link
Contributor Author

Thank you for your inform!!

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

Successfully merging this pull request may close these issues.

3 participants