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

[RA] Balance - Allied Barracks (Tent) cost increase #13930

Merged
merged 1 commit into from Aug 28, 2017

Conversation

Projects
None yet
4 participants
@SoScared
Member

SoScared commented Aug 28, 2017

Quick fix. Slipped through the code review of balance patch #13790

Allied Barracks, cost $500 up from $400

@SoScared

This comment has been minimized.

Show comment
Hide comment
@SoScared

SoScared Aug 28, 2017

Member

[18:40:17] < SoScared> forgot to add the barracks cost increase to the allied tent, only to soviet barracks
[18:40:39] < SoScared> cringy mistake but it happened

[18:57:53] <@orabot> New pull request #13930 by SoScared: [RA] Balance - Allied Barracks (Tent) cost increase | http://bugs.openra.net/13930

[19:04:13] < SoScared> pchote: ^ will #13930 be a pain to slip that through after the tag and pack?

[19:04:22] <+pchote> SoScared: yes
[19:04:32] < SoScared> ouch
[19:04:38] < SoScared> sorry about that
[19:04:53] <+pchote> the only option is to scrap the current playtest and make an entirely new one
[19:05:04] <+pchote> only option aside from shipping what we already have
[19:05:32] <+pchote> but i'm washing my hands of this - i've done all I need to for the next playtest, so this is between you and reaperrr/abc/whoever
[19:05:47] < SoScared> k k

@reaperrr @abcdefg30 any chance one of you (or any other) would have the time to redo the playtest as stated above? Pretty embarrassing mistake by me but it would be a shame shipping a playtest with such a glaring flaw.

Member

SoScared commented Aug 28, 2017

[18:40:17] < SoScared> forgot to add the barracks cost increase to the allied tent, only to soviet barracks
[18:40:39] < SoScared> cringy mistake but it happened

[18:57:53] <@orabot> New pull request #13930 by SoScared: [RA] Balance - Allied Barracks (Tent) cost increase | http://bugs.openra.net/13930

[19:04:13] < SoScared> pchote: ^ will #13930 be a pain to slip that through after the tag and pack?

[19:04:22] <+pchote> SoScared: yes
[19:04:32] < SoScared> ouch
[19:04:38] < SoScared> sorry about that
[19:04:53] <+pchote> the only option is to scrap the current playtest and make an entirely new one
[19:05:04] <+pchote> only option aside from shipping what we already have
[19:05:32] <+pchote> but i'm washing my hands of this - i've done all I need to for the next playtest, so this is between you and reaperrr/abc/whoever
[19:05:47] < SoScared> k k

@reaperrr @abcdefg30 any chance one of you (or any other) would have the time to redo the playtest as stated above? Pretty embarrassing mistake by me but it would be a shame shipping a playtest with such a glaring flaw.

@chrisforbes chrisforbes requested a review from reaperrr Aug 28, 2017

@Mailaender

This comment has been minimized.

Show comment
Hide comment
@Mailaender

Mailaender Aug 28, 2017

Member

OpenRA/OpenRAWeb#353 hasn't been merged yet, so there is still time to tag another playtest and update the newspost including download IDs.

Member

Mailaender commented Aug 28, 2017

OpenRA/OpenRAWeb#353 hasn't been merged yet, so there is still time to tag another playtest and update the newspost including download IDs.

@reaperrr

This comment has been minimized.

Show comment
Hide comment
@reaperrr

reaperrr Aug 28, 2017

Contributor

OpenRA/OpenRAWeb#353 hasn't been merged yet, so there is still time to tag another playtest and update the newspost including download IDs.

I guess that's what we should do then, this also gives us a little more time to hopefully have the organization issues regarding social media adressed by next weekend.

Contributor

reaperrr commented Aug 28, 2017

OpenRA/OpenRAWeb#353 hasn't been merged yet, so there is still time to tag another playtest and update the newspost including download IDs.

I guess that's what we should do then, this also gives us a little more time to hopefully have the organization issues regarding social media adressed by next weekend.

@SoScared

This comment has been minimized.

Show comment
Hide comment
@SoScared

SoScared Aug 28, 2017

Member

Thanks guys, sorry about the extra work.

Adding the fix ought to help getting more players over to the next playtest.

Member

SoScared commented Aug 28, 2017

Thanks guys, sorry about the extra work.

Adding the fix ought to help getting more players over to the next playtest.

@reaperrr reaperrr merged commit 1a95aed into OpenRA:bleed Aug 28, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment