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

Don't mark items as new if progression is disabled #752

Merged
merged 2 commits into from
Nov 3, 2023

Conversation

ASpoonPlaysGames
Copy link
Contributor

@ASpoonPlaysGames ASpoonPlaysGames commented Oct 21, 2023

closes #741

also doesn't show the challenge completion notifications in game as a nice side effect

To test disable progression and then "unlock" something (complete a weapon challenge or something)

@ASpoonPlaysGames ASpoonPlaysGames added needs testing Changes from the PR still need to be tested needs code review Changes from PR still need to be reviewed in code labels Oct 21, 2023
@itscynxx
Copy link
Contributor

I disabled progression, played for a bit, completed a challenge, and can't find any skin marked as new or anything, which as far as I understand is the intended result

Code also looks very simple and straight forward

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

@GeckoEidechse GeckoEidechse left a comment

Choose a reason for hiding this comment

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

Signing-off changes based on previous review. I did not test or code-review this PR myself.

@GeckoEidechse GeckoEidechse merged commit faff062 into R2Northstar:main Nov 3, 2023
3 checks passed
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.

Progression system incorrectly labels items as newly unlocked
3 participants