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

Subnautica: Rename location check (id 33029) #1120

Merged
merged 4 commits into from
Feb 26, 2023

Conversation

Klenoa
Copy link
Contributor

@Klenoa Klenoa commented Oct 20, 2022

What is this fixing or adding?

Updates the name of 'Grassy Plateaus West Wreck - Databox' to 'Grassy Plateaus Southwest Wreck - Databox'. This helps distinguish between the two distinct wrecks within the western grassy plateaus area and brings the name into alignment with the Poptracker Subnautica package.

How was this tested?

I did not perform any testing of this change, since at this time I am unsure how to build the Archipelago project from source and this was a relatively simple change to a JSON file name. If desired, I can learn how to build the Archipelago source and verify this JSON change works as intended.

Update southwest grassy plateaus wreck to 'Grassy Plateaus Southwest Wreck - Databox' instead of 'Grassy Plateaus West Wreck - Databox'
@black-sliver black-sliver added the is: enhancement Issues requesting new features or pull requests implementing new features. label Oct 27, 2022
@ThePhar
Copy link
Member

ThePhar commented Oct 28, 2022

Changing a location name will also require an increment of the data_version in the __init__.py file.

Copy link
Member

@Berserker66 Berserker66 left a comment

Choose a reason for hiding this comment

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

Can confirm it's a different wreck, but will need a datapackage bumb here and a json update in the mod.

Klenoa and others added 2 commits November 6, 2022 15:59
Description:
 + Update data_version to reflect the change in location name.
@Klenoa
Copy link
Contributor Author

Klenoa commented Nov 7, 2022

Thank you both for the feedback! Per your request, I have updated the data_version and have created a separate PR for the poptracker, which can be found here.

@Berserker66 I'm not entirely sure if those changes in the poptracker were what you were requesting (I believe the Crater.json file was correct already, barring the minor discrepancies in naming), so if I missed/overlooked something or misunderstood your request, let me know!

@Berserker66 Berserker66 merged commit 5042598 into ArchipelagoMW:main Feb 26, 2023
kindasneaki pushed a commit to kindasneaki/Archipelago that referenced this pull request Jun 28, 2023
Update southwest grassy plateaus wreck to 'Grassy Plateaus Southwest Wreck - Databox' instead of 'Grassy Plateaus West Wreck - Databox'
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants