-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[Enhancement][Challenge] Nuzlocke challenge #2421
Conversation
feat: Add link to Discord in menu UI feat: Add Discord and Google login functionality Add container around discord and google icons refactor: Update environment variable names for Discord and Google client IDs feat: Add "Or use" translation for login options in multiple languages feat: Update menu UI translations for multiple languages Code review fixes refactor: Update Discord and Google client IDs in environment variables
There's an existing Nuzlocke PR, but it was created before the addition of challenges. Maybe an opportunity for collaboration or superseding? |
That's Sam and he left. I already looked into the code but I can't use anything since everything changes so much and there is also so much other stuff like custom nuzlocke trainer etc. I will ask to close that old PR. And I am actually mostly done with rule logic because the challege system is pretty easy to use. |
Co-authored-by: Jannik Tappert <[email protected]>
… from regular challenge.
this also reduces output noise in tests
… a bug that lets random forbidden items spawn.
@sirzento What about this scenario? Opponent has REVIVER_SEED Player will take REVIVER_SEED from opponent. Should this be illegal in nuzlocke mode? |
@hayuna Yeah I thought about it and just thought it will be alright because it's just so rare to get it like that. I could disable it somehow if that's wanted but I think it will be alright like that. |
* Bug Fix * Update src/system/game-stats.ts Co-authored-by: flx-sta <[email protected]> * Fixing a test * Fixed import * Revert "Bug Fix" This reverts commit 8348449. --------- Co-authored-by: Frutescens <info@laptop> Co-authored-by: flx-sta <[email protected]>
…ames#3122) * Bug Fix * Update src/system/game-stats.ts Co-authored-by: flx-sta <[email protected]> * Fixing a test * Revert "Fixing a test" This reverts commit 1bdbe2d. * Test Re-Run --------- Co-authored-by: Frutescens <info@laptop> Co-authored-by: flx-sta <[email protected]>
…ch and double battles (pagefaultgames#2462) * Adjust how counter attacks target to account for uturn/voltswitch * Creates move flag for metal burst/comeuppance to redirect in some cases * Remove debug printing * Bit shifts the redirect counter flag * Removes extraneous class from prior testing * Remove vitest timestamp file that was accidentally added
Mystic ticket icons in the achievements just confuse players
…red in dex (pagefaultgames#2713) * Fix reins et al being available without unlocking second mon * Small fix * Parentheses for safety * Add documentation to new function
Co-authored-by: Frutescens <info@laptop>
…stats (pagefaultgames#3117) * Fixed OHKO moves being affected by accuracy and evasion battle stats * Added related tests for Fissure, unskipped related test for Hustle * Tweaked fissure accuracy and evasion tests to use spyOn() for getAccuracyMultiplier() as per feedback * Fixed accuracy test for Fissure
* [Fix][Sprite] Politoed Back Sprites Retrieved first version from history. Rearranged frames to make jump a little less floaty. Set same anim length for all variations. Json is identical for all variations. Spritesheet frame position is identical for all variations. Trimmed the frame of extra space Reduced file size of spritesheets and JSON due to less unique frames needed. * [Fix][Sprite] Politoed Back Sprites - Reformatted JSON to texturepacker standard Reformatted json.
* fix remove pokemon menu * fix dropdown all is not changing bug * fix bug when go down with no starter on start button * fix starter corsor bug on deletion. out of screen cursor bug * fix challenge log * fix lint error
… HA (pagefaultgames#3138) * Prevent evolution causing a swap from the second ability to the HA * Add tests * Update `starter-select-ui-handler.ts`
* More work * More work * Should be finished * Fixed an error in checking target * Moved white herb effect to StatChangePhase, 50% chance of consumption * Added graphics for White Herb * Balance and documentation * Add localization entry for White Herb apply msg * Add new keys to other localization files * German translations Co-authored-by: Jannik Tappert <[email protected]> * Update item sprite atlas * Redo item atlas * Remove whitespace in move.ts * Moved decrement outside conditional * Fix item atlas --------- Co-authored-by: Jannik Tappert <[email protected]>
Can you fix the PR and then split this into multiple discrete challenges (ie: permadeath challenge, no auto-heal challenge, etc)? |
@sirzento I'm putting this into draft while you resolve those conflicts and work on the open comments. Are you actually still actively working on this? |
Draft update at #4311 |
@DayKev so should this one be closed in favor of yours? |
Probably? I guess since the history is broken in this one it can't be used regardless. |
What are the changes?
This PR adds the challenge mod "Nuzlocke" to the game. As a result, various new challenge checks will be added that can also be used for other challenges in the future.
Why am I doing these changes?
I have seen some feature request for a nuzlocke challenge. Also, the old roadmap even had a nuzlocke mode as goal.
What did change?
The Nuzlocke challenge aswell as the achievement for it was added. I also changed the challenge UI and made the description field larger to be able to display more text.
5 new
challengeType
's where added to check for various rules in the nuzlocke challenge.Those types are:
NO_HEAL_PHASE
Checks if the heal phase after every 10th round should be activated.
SHOP_ITEM_BLACKLIST
Filters the shop for specific items.
RANDOM_ITEM_BLACKLIST
Filters the random item selection for specific items.
ADD_POKEMON_TO_PARTY
Checks if the current catched pokemon can be added to the party.
MOVE_BLACKLIST
Checks if move the user wants to use is illegal for this challenge.
Screenshots/Videos
Nuzlocke Regular:
Nuzlocke Hardcore:
How to test the changes?
Set
POKEBALL_OVERRIDE
to have some masterballs andMOVESET_OVERRIDE
to includeMoves.REVIVAL_BLESSING
.You can also set
OPP_LEVEL_OVERRIDE
to 1.Start a new challenge with Nuzlocke challenge enabled and check:
Checklist
npm run test
)