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

Pokemon Emerald: Change "settings" to "options" in docs #2517

Merged
merged 3 commits into from
Dec 28, 2023

Conversation

Zunawe
Copy link
Collaborator

@Zunawe Zunawe commented Nov 27, 2023

What is this fixing or adding?

Brings the Emerald docs in line with #2037.

How was this tested?

Ran web host, checked pages, clicked links.

If this makes graphical changes, please attach screenshots.

@ThePhar ThePhar added the is: documentation Improvements or additions to documentation. label Nov 27, 2023
Copy link
Contributor

@ReverM ReverM left a comment

Choose a reason for hiding this comment

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

There seems to be two instance of the word setting left in the info doc
one is at the line 31 (with a setting it can..) and the other is at line 46 (depending on your receive item message setting)

Other than that, looks good to me

Copy link
Contributor

@ReverM ReverM left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@Exempt-Medic Exempt-Medic left a comment

Choose a reason for hiding this comment

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

LGTM

@ReverM
Copy link
Contributor

ReverM commented Dec 20, 2023

I have covered the changes in #2622, maybe this PR could be closed then?

@Zunawe
Copy link
Collaborator Author

Zunawe commented Dec 20, 2023

This already has reviews and doesn't need to wait on the additional approvals of all other world maintainers, so I think it makes more sense to keep this one.

@alwaysintreble
Copy link
Collaborator

This pr is also from the world maintainer so takes precedence imo

@black-sliver black-sliver merged commit 04d194d into ArchipelagoMW:main Dec 28, 2023
7 checks passed
Jouramie pushed a commit to Jouramie/Archipelago that referenced this pull request Feb 28, 2024
…W#2517)

* Pokemon Emerald: Change "settings" to "options" in docs

* Pokemon Emerald: Fix two more usages of "setting" instead of "option"

* Pokemon Emerald: Minor rephrase in docs

Co-authored-by: Aaron Wagener <[email protected]>

---------

Co-authored-by: Aaron Wagener <[email protected]>
@Zunawe Zunawe deleted the emerald-docs-options branch June 1, 2024 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: documentation Improvements or additions to documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants