Roll20 uses cookies to improve your experience on our site. Cookies enable you to enjoy certain features, social sharing functionality, and tailor message and display ads to your interests on our site and others. They also help us understand how our site is being used. By continuing to use our site, you consent to our use of cookies. Update your cookie preferences .
×
Create a free account

[API] Regressions for Inline Dice Expression in the sendChat() API Function.

July 23 (9 years ago)

Edited July 23 (9 years ago)
The Aaron
Pro
API Scripter
I've been tracking down Inline Dice Expressions which crash the API, but work in chat (mostly -- [[[a] 1d1 ]] fails in chat as well.).

I wrote a script that tests variations of each of these rolls which can be used in validating fixes.  Here is the output (Automatically runs on('ready',...) ):


Here is the script:  https://github.com/shdwjk/Roll20API/blob/master/Di...

Thanks!
July 24 (9 years ago)
Riley D.
Roll20 Team
Okay, I'll definitely take a look at this. Thanks!
September 05 (9 years ago)
The Aaron
Pro
API Scripter
Updated with another failing test:


https://github.com/shdwjk/Roll20API/blob/master/DiceTests/DiceTests.js
September 21 (9 years ago)
Lucian
Pro
API Scripter
Is there any update on this? The unpredictable behaviour of rolls sent via the API is pretty frustrating right now and it's been 2 months since Aaron originally posted :-/
October 04 (9 years ago)
The Aaron
Pro
API Scripter
Updated with more failing cases, dealing with attribute expansion.  Additionally, it now creates a test character with test attributes and reports what the error was when it fails, as well as the value of the attribute in the test that failed.





https://github.com/shdwjk/Roll20API/blob/master/DiceTests/DiceTests.js
October 05 (9 years ago)
I have tested and verified this on my end.

@Lucian, we are working on an API update that should hopefully address a lot of the issue people are experiencing. I've marked this down on our list of things to address.

Thanks for all the work here Aaron.