Lstebner/experience pt2

lstebner commented 1 year ago

What this PR does

this is part 2 of the experience update outlined in #371

update the level progress indicator to be able to reference experience instead of items sold when not using the legacy system. to make this easier to do, this part of the Navigation component was broken out into its own file
added some constants to use for various experience values. these are just some initial ideas and definitely not set in stone
add achievement for paying off loan and getting an experience bonus for it
removing all references to the EXPERIENCE feature and legacy leveling system after some back and forth discussion in the comments here
updating all calls to levelAchieved to pass in experience
adding processLevelUp to addExperience so level ups are properly processed any time experience is modified
removing farmProductSalesNeededForLevel and replacing with experienceNeededForLevel along with any wording referencing "sales needed" to mention "experience points" instead

How this change can be validated

[x] start a new game with the experience feature enabled and check that the legacy leveling system toggle is "off" (in settings)
[x] load an existing save and check that the legacy leveling system toggle is "on"
[x] toggle the legacy leveling system on and off and check that the level indicator correctly displays "sales" or "experience" text
[x] pay off the initial debt and check that you are rewarded with an achievement and also experience

the easiest way to monitor an experience change is to either have the legacy leveling system off and just watch the level indicator progress bar, or have the component inspector open and watch the "experience" state value go up 📈

Questions or concerns about this change

n/a

Additional information

part 3 of this will be to hook up the rest of the experience rewards and do a QA pass on all of it. after that we can go live with this change!

when the legacy system is disabled the tooltip will indicate experience is required instead of sales:

achievement for debt-free:

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
farmhand	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 8, 2023 2:34pm

lstebner commented 1 year ago

alright another round of updates is in @jeremyckahn . i think we're getting there! i've been doing some testing as well and things are seeming pretty stable

jeremyckahn commented 1 year ago

I noticed a bug with online play: Peers' levels are no longer being reported correctly.

This is fixed in https://github.com/jeremyckahn/farmhand/commit/fdab492ace02ba2e2a9550549d6196b66330ba5a.

jeremyckahn / farmhand