MicroOcpp icon indicating copy to clipboard operation
MicroOcpp copied to clipboard

Getting error after upgrading to 0.1.0 (from 0.0.4) - startTransaction

Open shardul10 opened this issue 3 years ago • 6 comments

I recently upgraded to 0.1.0.

I see startTransaction now takes idTag as first argument. Is there any place where I can see changes that have happened between the versions, especially the ones not backward compatible.

shardul10 avatar Jul 23 '22 09:07 shardul10

Hello, what error does it give you?

stig888881 avatar Jul 24 '22 11:07 stig888881

Well, the startTransaction() now requires const char *idTag as first argument. In my old code (with 0.0.4), this function call did not require this argument. Just wondering if there any other changes documented that I should be aware of.

shardul10 avatar Jul 24 '22 13:07 shardul10

have you tried running the example ? it doesn't work for me. The server gives an error, there is no IdTagInfo field in the transaction request.

stig888881 avatar Jul 24 '22 16:07 stig888881

@shardul10 In version 0.0.4, startTransaction(...) had no dedicated idTag parameter but used the cached idTag from a previous Authorize operation. But in case you authorize your idTags locally, that leads to a superfluous message and after all, it didn't really simplify the transaction process. Apart from that and refactoring some data types, I didn't change the functionality of the existing functions.

@stig888881 Can you try capturing the console output when triggering a StartTransaction? Would be helpful, thanks!

matth-x avatar Jul 24 '22 21:07 matth-x

Thanks @matth-x. Just wanted to be sure there is no other changes. I did change my code to pass idTag to startTransaction and it works.

shardul10 avatar Jul 25 '22 03:07 shardul10

have you tried running the example ? it doesn't work for me. The server gives an error, there is no IdTagInfo field in the transaction request.

Yes, the example works.

shardul10 avatar Jul 25 '22 03:07 shardul10

Closing this due to inactivity

matth-x avatar Aug 25 '23 10:08 matth-x