nicholaskillin / gloomhaven-hand-manager Goto Github PK

Right now if you confirm perks, then go back, then confirm perks again, you get double the enhancement spots that you should

gasgsa

Love the app! Only thing that is missing is the Health, gold and experience that is part of the original. I am assuming you are going to be adding it, but did have an idea of when that would be completed? Looking to make this our main gloomhaven app!

Does this work?

It would be great to unselect trackers on an active card, if the option was selected in error. Also, it is possible to select higher numbers before lower ones, which is a little odd. Maybe a +/- option to increment or decrement the tracker?

https://docs.bugsnag.com/platforms/javascript/react/

When an enhancement is chosen, a generic "Enhancement Name" displays on the associated card. Otherwise no impact on play.

When long resting, you are told to Choose One Card to Lose. Selecting a card on the top row puts a red selection border around the card. Selecting a card that is not on the top row does not show the boarder. However, it will correctly Lose the card that you had chosen. Tested with a Brute for a 2 row discard pile and a tinkerer for a 3 row pile.

On the hosted version of the app the file names are case sensitive, so a lot of the class attack modifiers just show broken links. Need to make sure that the URL's called match the cases of the files.

ggaew

Button for losing a card to damage does not do anything

One quick piece of feedback from someone with a slower internet connection: While the images are loading the locked classes show the name of the class. Would be better to replace that with the spoiler safe name.

ststes

This is such a great upgrade to the previous build of this manager. My friends and I have been doing gloomhaven remotely (after a couple of in person sessions prior to the pandemic) and we have been using the old setup. One thing that we do notice that is missing is the bottom section of the previous build, where you can add status effects and also track your health, experience and loot. If those sections could be added back in, this would be the only thing we would need for remote play! Thanks for all the hard work and look forward to these being added back in.

Some kind of a header/footer/alert to let people know how to reach out if they run into any issues. Include link to github and https://boardgamegeek.com/geekmail/compose?touser=nicholaskillin.

To reproduce:

1. Select a character and select all of the perks. Confirm that.
2. Refresh the page.
3. Head back to the select perks page.
4. Uncheck some perks.

What I noticed: The modifications state isn't changing. So even though boxes are being unchecked, all of the modifier deck modifications are still being applied.

The last perk is incorrect.

What it currently says

Ignore Negative Scenario Effects

What it should say

Ignore negative item effects

Sometimes you put some cards in the Chosen area and then change your mind before the round begins. The only way to correct that for now is to discard both of them, and then recover them from there.

If I add this back in, you would have me to be able to turn it off. Gloomhaven Helper does this already and it makes more sense for these things to live there.

However it wouldn’t be bad to make this an option for groups not using Gloomhaven Helper, I would just want it to be an opt in thing that you can turn on if you want.

fsmflpsafm

Currently, it says your hand size is full, when trying to select more cards than you can bring along. It would be awesome, if it instead allows it and perhaps make the border for all cards red (and make it so you can't continue)

This allows you to select all the cards you consider, and then cut down to what is allowed :)

Nice app :)

The variable for those two characters has a capital letter in the middle of the name, but the name being passed from getCharacter() is all lowercase. Need to make the two match up.

Needed an array