This is still a problem. main webpage returns html and does not return json, so this change will still cause the error and is not an acceptable fix.
I hope you can take the generated patch and use the patch command to apply on top of the existing trunk, no? or may be just yet, use git apply the patch stub? (it comes with nice variables and syntax style consistency as a bonus)
git apply patch will take care of the patches to the files mentioned in the patch file…
nevermind, I understand you need a pull request, supposedly that will be simple.
I will put things together including the method for this feature request ticket as well, and raise a github PR that will be much easier I suppose for you to review/check/test and merge (but for that I am supposed to restore my long forgotten github account
@dexter , you are right about the definition about volume, it is total number of shares traded, however what i want is live total number of trades executed. ( lets say i buy 15 lakh shares in 3 transactions - 5L, 5L, 5L) so the total volume would be 15 lakh and total number of trades would be 3(example).