search

toconnell / kdm-manager

An interactive campaign manager for the game "Monster", by Kingdom Death. Development blog and release notes at https://blog.kdm-manager.com This project has no affiliation with Kingdom Death and is a totally independent, fan-maintained project.
http://kdm-manager.com
Other
26 stars 11 forks source link

Survivor Sheet: survivor attribute toggles malfunctioning #532

Closed toconnell closed 5 years ago

toconnell commented 5 years ago

when adding to the courage/understand/proficiency I have to do it 2 or 3 times before it actually takes.

Can't replicate this one yet: waiting for some clarification.

toconnell commented 5 years ago

This is happening on the Survivor Sheet (not the Campaign Summary). Working it now.

toconnell commented 5 years ago

It looks like a timing issue on the JS...it seems like the change is making it to the API, but the view is pulling the survivor down before the change is saved, etc.

toconnell commented 5 years ago

This actually only happens when you're also setting one of the A&Is that goes with the attribute, e.g. "Stalwart" or "Tinker" or whatever.

Looks like we can just have postJSONtoAPI return us a promise and then do the update after we get the promise back. Should fix us up.

toconnell commented 5 years ago

Looks like that worked. Rolling this one out in the next deploy of the legacy webapp.