Skip to content

refactor: change to use new msu affordability preview system#352

Merged
LordMidas merged 6 commits intodevelopmentfrom
refactor-change-to-new-msu-affordability-system
May 14, 2024
Merged

refactor: change to use new msu affordability preview system#352
LordMidas merged 6 commits intodevelopmentfrom
refactor-change-to-new-msu-affordability-system

Conversation

@LordMidas
Copy link
Copy Markdown
Contributor

@LordMidas LordMidas commented Apr 18, 2024

This is based on the new affordability system I am implementing in MSU.

EDIT: 2024-05-10
I've created an even better affordability system in MSU so the PR has been updated to account for that.

@LordMidas LordMidas added this to the 0.5.0 milestone Apr 18, 2024
@LordMidas LordMidas requested a review from Darxo April 18, 2024 05:27
LordMidas added 3 commits May 9, 2024 23:28
Instead of comparing preview values. I realized that it is ok to use these functions because they are based on CurrentProperties and we don't mess with that during update functions.
@LordMidas LordMidas force-pushed the refactor-change-to-new-msu-affordability-system branch from 30677a7 to 85789ca Compare May 10, 2024 04:02
@LordMidas LordMidas requested a review from Darxo May 10, 2024 04:02
@LordMidas
Copy link
Copy Markdown
Contributor Author

LordMidas commented May 10, 2024

I've created an even better affordability system in MSU so the PR has been updated to account for that. Please re-review based on the changes in the latest commit.

Comment thread scripts/skills/perks/perk_rf_fresh_and_furious.nut
Comment thread scripts/skills/perks/perk_rf_vanquisher.nut
Comment thread scripts/skills/actives/rf_en_garde_toggle_skill.nut Outdated
Comment thread scripts/skills/perks/perk_rf_fresh_and_furious.nut
Comment thread scripts/skills/perks/perk_rf_momentum.nut
Comment thread scripts/skills/perks/perk_rf_vigorous_assault.nut
@LordMidas LordMidas merged commit c2417df into development May 14, 2024
@LordMidas LordMidas deleted the refactor-change-to-new-msu-affordability-system branch May 14, 2024 02:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants