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

Shivers: Add collect behavior option. #3854

Merged
merged 2 commits into from
Aug 31, 2024

Conversation

korydondzila
Copy link
Contributor

What is this fixing or adding?

This adds a new option that has not affect on generation and is purely for client side behavior.
This enables the player to select how collection affects their world.

How was this tested?

I checked that the player option on webhost had the new options

If this makes graphical changes, please attach screenshots.

Screenshot 2024-08-29 at 22 58 36

@github-actions github-actions bot added the waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. label Aug 30, 2024
@NewSoupVi
Copy link
Member

NewSoupVi commented Aug 30, 2024

This is fine of course, but did you consider just putting this into the client somehow? I made it a dropdown in my client & my players really appreciate being able to switch collect behaviors mid-game (or like, mid-seed - they do have to relaunch the client, but yeah)

@Exempt-Medic Exempt-Medic added the is: enhancement Issues requesting new features or pull requests implementing new features. label Aug 30, 2024
@korydondzila
Copy link
Contributor Author

korydondzila commented Aug 30, 2024

@NewSoupVi I took the time to get feedback and poll players over 2 weeks and this is something that can get changed in the future if people end up wanting the flexibility to change on the fly.

Co-authored-by: Scipio Wright <[email protected]>
@GodlFire
Copy link
Collaborator

World Maintainer: Looks good.

Copy link
Collaborator

@ScipioWright ScipioWright left a comment

Choose a reason for hiding this comment

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

Would also prefer it be a client option, but it's fine as is

@ScipioWright ScipioWright 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. labels Aug 30, 2024
@NewSoupVi NewSoupVi merged commit 8ed466b into ArchipelagoMW:main Aug 31, 2024
17 checks passed
@korydondzila korydondzila deleted the collect-behavior branch August 31, 2024 18:57
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.

5 participants