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

The 'disable infamy' checkbox actually disables infamy now! (also some other enhancements/major overhauls/ADHD moments) #76

Merged
merged 12 commits into from
Aug 3, 2024

Conversation

11BelowStudio
Copy link
Contributor

Okay so

  • the 'disable infamy' checkbox can be used to disable infamy (updates unlock cost of tier 4 appropriately, and will forcibly remove/refund skills which are now unobtainable accordingly)
  • 100% backward-compatible
  • can be toggled on the fly and it Won't Break

also some other changes:

  • Obtaining Iron Man aced will now automagically equip the ICTV (seeing as, y'know, that's the whole point of Iron Man aced)
  • Switching to a perk deck/copycat boost which unlocks a certain throwable now automagically equips said throwable for you
  • Switching copycat boosts to one without a throwable will now unequip your current throwable if it's now locked
  • readme now links to the actual netlify page (https://pd2builder.netlify.app) (and also to the github-pages instance)
  • updated my info in the contributors.md thing
  • Big rework of the 'stats.js' stuff as well
    • hp stat (and modifiers) are now shown
    • copycat boosts do things
    • somewhat completely reworked the stats info in the json files
    • but like it works!

happy 4:14 am

11BelowStudio and others added 12 commits August 1, 2024 10:28
toggling infamy thing will cause future updates on subtrees to behave in desired way - but won't update current subtree status. need to fix that.
I may have bastardized quite a bit of the backend code beyond all recognition - but it appears to work now.
(also changing the perk deck unlock thing will automatically select the newly-unlocked throwable for you (and deselect any newly-locked throwables))
…g as that's the whole point of the aced version of that perk)
Started off just simply updating the netlify link in the readme to point to the .app TLD instead of .com TLD (as .com doesn't work), and also updating my info in the CONTRIBUTORS.md

Then I got mildly sidetracked by thinking 'hmm maybe I should replace some deprecated functions eslint is complaining about with non-deprecated functions'

And then I found out about stats.js and uhhhhhhh yeah massive rewrite but now it shows hp stats as well and the copycat perks function too

(now that's what I call a certified ADHD moment)
@ImUrX ImUrX merged commit 507041a into r-paydaybuilds:master Aug 3, 2024
6 checks passed
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