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: Add event ticket locations to client data store flags #3177

Merged
merged 3 commits into from
May 5, 2024

Conversation

Zunawe
Copy link
Collaborator

@Zunawe Zunawe commented Apr 19, 2024

What is this fixing or adding?

Adds the event ticket locations to the pokemon_emerald_keys_ data storage key so that they can be autotracked even if they're not randomized.

This pushes the key over 32 bits, but that's probably not an issue for anyone.

How was this tested?

It wasn't except to run the launcher, which should verify that these keys have location data. I'll hand this PR over to the tracker devs to verify.

@github-actions github-actions bot added the waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. label Apr 19, 2024
@Zunawe Zunawe added the is: enhancement Issues requesting new features or pull requests implementing new features. label Apr 19, 2024
@Zunawe
Copy link
Collaborator Author

Zunawe commented Apr 21, 2024

Added an event flag for the pokemon_emerald_events_ key, since it's a small, related change.

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.

Merged into main, generated and connected.

@Exempt-Medic Exempt-Medic added waiting-on: core-review Issue/PR has been peer-reviewed and is ready to be merged or needs input from a core maintainer. waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. waiting-on: other Issue/PR is waiting for something else, like another PR. and removed waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. waiting-on: core-review Issue/PR has been peer-reviewed and is ready to be merged or needs input from a core maintainer. labels Apr 22, 2024
@Exempt-Medic
Copy link
Member

Exempt-Medic commented Apr 22, 2024

I added waiting-on: other since you said "I'll hand this PR over to the tracker devs to verify." getting some kind of update on how that goes would be good

@Zunawe
Copy link
Collaborator Author

Zunawe commented Apr 30, 2024

Added a few more flags at the request of the tracker devs

@Zunawe Zunawe force-pushed the emerald-event-ticket-flags branch from 5b32c6e to d48ac26 Compare April 30, 2024 01:21
@vyneras
Copy link

vyneras commented May 4, 2024

Tracker dev here, I have verified that that these changes worked for autotracking the new events and key items

@Exempt-Medic Exempt-Medic added waiting-on: core-review Issue/PR has been peer-reviewed and is ready to be merged or needs input from a core maintainer. and removed waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. waiting-on: other Issue/PR is waiting for something else, like another PR. labels May 4, 2024
@NewSoupVi NewSoupVi merged commit d94cf8d into ArchipelagoMW:main May 5, 2024
16 checks passed
@Zunawe Zunawe deleted the emerald-event-ticket-flags branch May 23, 2024 08:42
jnschurig pushed a commit to Tranquilite0/Archipelago-SoulBlazer that referenced this pull request Jun 13, 2024
ArchipelagoMW#3177)

* Pokemon Emerald: Add event ticket locations to client data store flags

* Pokemon Emerald: Add regi doors event flag

* Pokemon Emerald: Add more tracker flags
qwint pushed a commit to qwint/Archipelago that referenced this pull request Jun 24, 2024
ArchipelagoMW#3177)

* Pokemon Emerald: Add event ticket locations to client data store flags

* Pokemon Emerald: Add regi doors event flag

* Pokemon Emerald: Add more tracker flags
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: enhancement Issues requesting new features or pull requests implementing new features. waiting-on: core-review Issue/PR has been peer-reviewed and is ready to be merged or needs input from a core maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants