capturetheflag
capturetheflag copied to clipboard
give score for items removed from enemies
This PR gives score to the players if they put an item dropped from teammate or enemy and put it in the teamchest. The item must not be stackable.
What stops players from just cashing in all the items they've collected and talking it all back from the chest as soon as they get the points?
What stops players from just cashing in all the items they've collected and talking it all back from the chest as soon as they get the points?
It's a reasonable concern. But why they would do that?
To play it safe I would prefer to only give 1 or less score for items we reward returning, and slowly increase that as we watch things play out on the main server
To play it safe I would prefer to only give 1 or less score for items we reward returning, and slowly increase that as we watch things play out on the main server
I disagree with the reward being the same, one point, for all the items. And less than one is a bad idea IMO
I disagree with the reward being the same, one point, for all the items. And less than one is a bad idea IMO
Did you read the and slowly increase that as we watch things play out on the main server part? The item values can be differentiated again as we increase them
I disagree with the reward being the same, one point, for all the items. And less than one is a bad idea IMO
Did you read the
and slowly increase that as we watch things play out on the main serverpart? The item values can be differentiated again as we increase them
Yeah I know but it doesn't make sense to me even temporary. But if you mean some stuff can be temporary non-sensible, that's your decision. I am committing to give exactly one for all items.
Thanks. https://github.com/MT-CTF/capturetheflag/pull/1173#discussion_r1254898943 needs to be fixed and this will be ready for testing
Your rebase didn't work at all there
With rebasing I tend to use git rebase -i <PR branch>~<number of previous commits I want to have the option to squash> (e.g git rebase -i master~6), squash all my commits together, then git pull -r mainrepo master to rebase off of the CTF repo's master branch
I'm laying that out to show you some things you can do, I highly recommend googling/figuring out what those commands do on your own though, you can seriously mess things up otherwise
Items returned to team chest shouldn't give score if they're your own
@LoneWolfHT Done.
Rebase conflict
WDYM?
@LoneWolfHT Done
How are you rebasing your PR? This shouldn't be showing up:
@LoneWolfHT What was state of this? Do I need to do anything?