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

GMPE01, GP5E01, GP6E01, GP7E01, RM8E01: Code Updates #11734

Merged
merged 1 commit into from Apr 18, 2023

Conversation

EndangeredNayla
Copy link
Contributor

No description provided.

@AdmiralCurtiss
Copy link
Contributor

Please explain what these updates are for.

@EndangeredNayla
Copy link
Contributor Author

This pr addresses code changes made in the mario party netplay community.. codes were made to improve gameplay.

@JMC47
Copy link
Contributor

JMC47 commented Apr 11, 2023

I wasn't able to test all of the cheat codes, but they seem to be properly added and credited.

Copy link

@Rumi-Larry Rumi-Larry left a comment

Choose a reason for hiding this comment

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

A bunch of names for cheats have abnormal grammar, that needs checking by a native English speaker.
Also, I recommend that all instances of "Last 5 Turns" be between parentheses and with the word "events" appended e.g. ["Last 5 Turns" events happen....]. Otherwise it's very confusing to read.

Data/Sys/GameSettings/GP5E01.ini Outdated Show resolved Hide resolved
Data/Sys/GameSettings/GP5E01.ini Outdated Show resolved Hide resolved
Data/Sys/GameSettings/GP7E01.ini Show resolved Hide resolved
Data/Sys/GameSettings/GP7E01.ini Outdated Show resolved Hide resolved
Data/Sys/GameSettings/GP5E01.ini Outdated Show resolved Hide resolved
Data/Sys/GameSettings/GP7E01.ini Outdated Show resolved Hide resolved
@EndangeredNayla
Copy link
Contributor Author

any chance for a review?

@AdmiralCurtiss AdmiralCurtiss merged commit 4d1864e into dolphin-emu:master Apr 18, 2023
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants