datdotorg / datdot-service

datdot service
https://playproject.io/datdot-service
MIT License
15 stars 9 forks source link

use isInBlock instead of isFinalized #4

Closed jam10o-new closed 4 years ago

jam10o-new commented 4 years ago

what was previously named "isFinalized" is now named "isInBlock" - (isFinalized now actually triggers on finality, not block inclusion), we probably want to respond to events when they are isInBlock, not isFinalized.

super-long-term, a more robust mechanism would wait for finality and retry actions if finality is not achieved before dropping the extrinsic callback.

ninabreznik commented 4 years ago

Ok, so I am pulling new version of datdot-substrate, building it and then changing in datdot-service isFinalized to isInBlock and will first try if all still works. If yes, I continue with adding js part for the attestation.