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

Quickfix in wrong attacker receiving progression stat updates #755

Merged
merged 1 commit into from
Nov 3, 2023

Conversation

Zanieon
Copy link
Contributor

@Zanieon Zanieon commented Oct 22, 2023

Currently when assisting players with Titan kills the Assist Stat Tracking is still tracking to attacker, this took me a while to notice only because recently I resetted my whole persistent data to ensure everything is tracking properly for progression, and this popped in.

@Zanieon Zanieon added the needs code review Changes from PR still need to be reviewed in code label Oct 22, 2023
Copy link
Contributor

@ASpoonPlaysGames ASpoonPlaysGames left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good, simple bug fix.

@ASpoonPlaysGames ASpoonPlaysGames added READY TO MERGE This mergeable right now and removed needs code review Changes from PR still need to be reviewed in code labels Oct 22, 2023
@ASpoonPlaysGames
Copy link
Contributor

I'm not going to enforce testing for this PR, its such a simple change that you can tell that it'll work based on the surrounding logic

@ASpoonPlaysGames ASpoonPlaysGames changed the title [Progression Stats] Quickfix in wrong attacker recieveing stat updates [Progression Stats] Quickfix in wrong attacker receiving stat updates Oct 28, 2023
@GeckoEidechse GeckoEidechse changed the title [Progression Stats] Quickfix in wrong attacker receiving stat updates Quickfix in wrong attacker receiving progression stat updates Nov 3, 2023
@GeckoEidechse
Copy link
Member

Trusting @ASpoonPlaysGames on this one.

@GeckoEidechse GeckoEidechse merged commit 54fae18 into R2Northstar:main Nov 3, 2023
4 checks passed
@Zanieon Zanieon deleted the assist_quickfix branch November 3, 2023 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
READY TO MERGE This mergeable right now
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants