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

DS3: move an items location from RC -> DH #2017

Merged
merged 3 commits into from
Jul 23, 2023

Conversation

Br00ty
Copy link
Contributor

@Br00ty Br00ty commented Jul 23, 2023

What is this fixing or adding?

"Ring of Steel Protection+3" actually belongs in DH instead of RC. this will shift the item ID's for the last 3 items in RC, and should not shift any ids in DH

How was this tested?

very quickly with some generations

If this makes graphical changes, please attach screenshots.

"Ring of Steel Protection+3" actually belongs in DH instead of RC. this will shift the item ID's for the last 3 items in RC, and should not shift any ids in DH
@el-u
Copy link
Collaborator

el-u commented Jul 23, 2023

If this changes the datapackage, increment data_version to not break older clients?

@ThePhar
Copy link
Member

ThePhar commented Jul 23, 2023

If this changes the datapackage, increment data_version to not break older clients?

It looks like it's just being moved rather than changed so the contents of datapackage should be the same (checksum might update, but wouldn't matter for clients that still rely on version). I'll have to verify later.

@ThePhar ThePhar added the is: refactor/cleanup Improvements to code/output readability or organizization. label Jul 23, 2023
@Br00ty
Copy link
Contributor Author

Br00ty commented Jul 23, 2023

If this changes the datapackage, increment data_version to not break older clients?

just did this!

@black-sliver black-sliver merged commit cf8ac49 into ArchipelagoMW:main Jul 23, 2023
12 checks passed
@Br00ty Br00ty deleted the ds3-fix-location branch July 29, 2023 20:20
FlySniper pushed a commit to FlySniper/Archipelago that referenced this pull request Nov 14, 2023
* moves items location from RC -> DH

"Ring of Steel Protection+3" actually belongs in DH instead of RC. this will shift the item ID's for the last 3 items in RC, and should not shift any ids in DH

* updated data_version to 7
kl3cks7r pushed a commit to kl3cks7r/Archipelago that referenced this pull request Dec 15, 2023
* moves items location from RC -> DH

"Ring of Steel Protection+3" actually belongs in DH instead of RC. this will shift the item ID's for the last 3 items in RC, and should not shift any ids in DH

* updated data_version to 7
Jouramie pushed a commit to Jouramie/Archipelago that referenced this pull request Feb 28, 2024
* moves items location from RC -> DH

"Ring of Steel Protection+3" actually belongs in DH instead of RC. this will shift the item ID's for the last 3 items in RC, and should not shift any ids in DH

* updated data_version to 7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: refactor/cleanup Improvements to code/output readability or organizization.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants