Issue #330: don't log params for sync apply and update test

OdyseeTeam / odysee-api

API server for Odysee

https://odysee.com

MIT License

228 stars 49 forks source link

Issue #330: don't log params for sync apply and update test #348

Closed jbshu4546 closed 3 years ago

jbshu4546 commented 3 years ago

New PR here. I wasn't sure if my test change was the correct one, but it does pass. Please advise.

jbshu4546 commented 3 years ago

Pushed a change.

anbsky commented 3 years ago

This seems to be doing the job. But to make sure, it would be great to have a test for it. I would also appreciate cleaner commit history.

jbshu4546 commented 3 years ago

Roger. Test up next.

jbshu4546 commented 3 years ago

I opted to add asserts to the existing caller test.

anbsky commented 3 years ago

Thank you @jbshu4546, it's deployed and running in production now. Please send us your LBC wallet address so we can compensate you for the contribution.