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

More base presets #223

Merged
merged 7 commits into from
Feb 4, 2024
Merged

Conversation

SuperNerdEric
Copy link
Contributor

I went back and forth on the names and what should be in them for a while. I'm open to change them or add more.

Copy link
Collaborator

@LlemonDuck LlemonDuck left a comment

Choose a reason for hiding this comment

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

i'm also debating whether zenytes are a good idea here, but probably not. let's keep them as furies

@SuperNerdEric
Copy link
Contributor Author

@LlemonDuck @jayktaylor We want to start adding weapons to these or nah? See conversation in Discord

@LlemonDuck
Copy link
Collaborator

LlemonDuck commented Feb 3, 2024

No weapons for now. Let's keep it consistent with the other non-Barrows sets and we could revisit after some discussion if people really want default weapons equipped. I think it's better served by #185 / #217

@LlemonDuck LlemonDuck added the ui Issue affects the UI/React code label Feb 3, 2024
Copy link
Member

@jayktaylor jayktaylor left a comment

Choose a reason for hiding this comment

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

Happy to defer to @LlemonDuck for approving this one. Generally we probably don't want to make the preset list too long as to not scare the end user with too many options, but if these presets seem genuinely helpful, then I'm not opposed.

@jayktaylor jayktaylor merged commit 1f28eca into weirdgloop:main Feb 4, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ui Issue affects the UI/React code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants