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

Gat Cut #43

Merged
merged 2 commits into from
Jul 21, 2023
Merged

Gat Cut #43

merged 2 commits into from
Jul 21, 2023

Conversation

dvir001
Copy link
Contributor

@dvir001 dvir001 commented Jul 17, 2023

Cut the gat

About the PR

The Gut was Cut

Cut the gat
@Cheackraze
Copy link
Member

hm not ideal, they will just inherit the base staticprice of 20 this way and be worth even more rofl. If anything lower it to 1 or something idk

really we just cant let this kind of entity spam happen to the detriment of the server. This has been a fundamental problem with botany in vanilla SS14 since the dawn of the job role sadly.

@mirddes

This comment was marked as abuse.

@dvir001
Copy link
Contributor Author

dvir001 commented Jul 20, 2023

hm not ideal, they will just inherit the base staticprice of 20 this way and be worth even more rofl. If anything lower it to 1 or something idk

really we just cant let this kind of entity spam happen to the detriment of the server. This has been a fundamental problem with botany in vanilla SS14 since the dawn of the job role sadly.

Set to 1, please merge before they crash the server again.

@mirddes

This comment was marked as abuse.

@Cheackraze Cheackraze merged commit 9aa978f into new-frontiers-14:master Jul 21, 2023
7 of 12 checks passed
@sTiKyt
Copy link
Contributor

sTiKyt commented Jul 21, 2023

But how would that help with crashes? People can still grow tons of those and if they explode server crashes, right?

Maybe some way of additional stacking would make sense

For example, allow stack on the ground be of any size, so no matter how many thousands of same plant are just in 1 entity

But disallow picking that entire stack up, and pick up 30(or whatever max stack size is supposed to be) instead

@dvir001 dvir001 deleted the Gat-Cut branch July 21, 2023 08:58
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