-
Notifications
You must be signed in to change notification settings - Fork 565
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
Cas sentryguns pricedown and dinamic one for omnisentryguns #4909
Cas sentryguns pricedown and dinamic one for omnisentryguns #4909
Conversation
@@ -29,6 +32,7 @@ | |||
/obj/structure/machinery/part_fabricator/dropship/ui_data(mob/user) | |||
return list( | |||
"points" = get_point_store() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing ,
did you test this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did some untested changes on it recently, trying to finde out how UI works. it is draft for a reason
Why not make Alamo sentries go up in price too? And yeah, huge price is one way to balance 3 sentries inside Alamo, and you just made it a lot easier without any downsides. Also they are pretty strong, saying they have no impact is just wrong. |
|
|
This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself |
…or-omnisentryguns
Thank you for the PR, but I am currently not interested in the majority of the changes here. I would be alright with a lowering of the ground sentries price by a bit (100-200). |
About the pull request
Lowers price of normal sentryguns, they just are not worth 500 points each. Lowers price of launchbay to 200 from 400, why have it so expensive when there is no reason to ever buy second one. And for the major change, makes omnisentryguns price scale with how many you bought, they start at 300 and go up 100 each time you buy one, so lets do some math. First one is for 300, easy. If you buy 5 they are 500 points each, nice. When you buy 10 they are 800 each aka the same as now, and if you try to buy more they will be hella more expensive. This is here to help those who think this will cause sentry spam, it is more against it, then what we have now.
Explain why it's good for the game
This gives marines another tool to use, most pilot will buy at least 1 omni sentry with this PR, most will not buy more then 3 or 5.
500 for alamo sentry is just noobtrap basicly, pilots just feel like they are griefing cas when they buy them, and they basically have no impact, if you are agains 3 sentryguns inside alamo, balance that, do not just make it expensive.
Basically, tool for marines to use in game = good.
Testing Photographs and Procedure
Screenshots & Videos
Put screenshots and videos here with an empty line between the screenshots and the
<details>
tags.Changelog
🆑
balance: transport sentrygun 500 -> 200
balance: launchbay 400 -> 200
balance: omnisentrygun 800->300, but they go up 100 in price each time you buy one
/:cl: