Stexinator / TheWitcherTRPG

FoundryVTT The witcher trpg system
GNU General Public License v3.0
10 stars 5 forks source link

fix diagrams not accepting associated items #123

Closed Stexinator closed 3 weeks ago

Stexinator commented 3 weeks ago

closes https://github.com/Stexinator/TheWitcherTRPG/issues/117 https://github.com/Stexinator/TheWitcherTRPG/issues/124 https://github.com/Stexinator/TheWitcherTRPG/issues/115

Stexinator commented 3 weeks ago

@pedroaugustobt @moliloo @AlexanderTheGr3at I found a game breaking regression so I want to release it today. Would be nice if you could do some tests

moliloo commented 3 weeks ago

I will test today, sorry for desapegar and dont make teste last week, i was really busy with college and home moving

Stexinator commented 3 weeks ago

I will test today, sorry for desapegar and dont make teste last week, i was really busy with college and home moving

No worries, every testing is helpful and with 3 testers there is mostly someone that got time :) Also I test all of the stuff when developing the features. But sometimes things slip or my data is already more advanced then the real user data out there :)

pedroaugustobt commented 3 weeks ago

@pedroaugustobt @moliloo @AlexanderTheGr3at I found a game breaking regression so I want to release it today. Would be nice if you could do some tests

Of course, I'll download the version and test it!

AlexanderTheGr3at commented 3 weeks ago

@pedroaugustobt @moliloo @AlexanderTheGr3at I found a game breaking regression so I want to release it today. Would be nice if you could do some tests

Everything seems to be working well for me. Only errors I'm getting are depreciation errors. I tested all the global modifiers, crafting and alchemy, attacks with the new compendium.

localhost-1717608324438.log localhost-1717608715458.log

AlexanderTheGr3at commented 3 weeks ago

Checked the new spell push. I just did a basic test with the default Mage and everything seems to be in order for spells.

SpellTest