-
Notifications
You must be signed in to change notification settings - Fork 6
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
Bugfix for Sequence Customization #1516
Conversation
67dc1ef
to
1b4a1b4
Compare
Tested and confirmed this version still fixes the bug. Agreed that the type change ought to be part of a bigger audit (and probably ought to feed into #1499 anyway). |
Latest commit ensures that the output is a string using string interpolation |
9a508db
to
fa00a28
Compare
fa00a28
to
87baa3a
Compare
6690d15
to
057d24e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm! I'll let @cartermak give the final say though.
Tested and working for me! Thanks all |
Opening #1514 as a branch rather than a fork
Small fix that probably came in when the interface changed from mutable
SeqJSON
to stringified JSON.Closes #1515