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

refactor: improve consumeables logic even more #68

Merged
merged 3 commits into from
Feb 12, 2024
Merged

Conversation

TonybynMp4
Copy link
Contributor

Description

Improves the consumeables logic more,
Added config for a few things:
animation / prop,
stress relief,
sustenance,
Alcohol level (how drunk you get)

Also fixed how the random worked, setting a random value in config WILL NOT make a new value everytime..

Also changed to use client callback instead of client events, sounds better to me, might be wrong tho.

Needs testing, but it's way too late for me to do that lol

Checklist

  • I have personally loaded this code into an updated Qbox project and checked all of its functionality.
  • My pull request fits the contribution guidelines & code conventions.

@mafewtm
Copy link
Member

mafewtm commented Feb 12, 2024

Can we just rename the stupid consumable table names? Idk why they made it ConsumablesEat and whatnot

@DevX32
Copy link

DevX32 commented Feb 12, 2024

Can we just rename the stupid consumable table names? Idk why they made it ConsumablesEat and whatnot

that would be better

@TonybynMp4
Copy link
Contributor Author

Can we just rename the stupid consumable table names? Idk why they made it ConsumablesEat and whatnot

What would you rename them to 🤔

@BerkieBb
Copy link
Contributor

Combine them into a consumables table and give them each a type

@TonybynMp4
Copy link
Contributor Author

TonybynMp4 commented Feb 12, 2024

Combine them into a consumables table and give them each a type

Berkie ily smart guy

@TonybynMp4
Copy link
Contributor Author

I hate annotations.

@mafewtm mafewtm merged commit 1514054 into main Feb 12, 2024
4 checks passed
@TonybynMp4 TonybynMp4 deleted the consumeables branch February 12, 2024 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants