starknet-io
Typed data aggregate (SNIP-12)
Motivation and Resolution
Aggregate for the reported typed data issues. Each commit addresses a corresponding issue (or PR).
Extracted to provide a unified overview so that potential impact can be evaluated and coordinated.
Issues:
- #1278
- #1286
- #1341
- #1342
- #1343
- #1348
- #1039
Summary of what we currently plan, please ack
@xJonathanLEI @sgc-code @franciszekjob @penovicp @tabaktoni @thiagodeev
#1278 - fix js implementation #1286 - fix js implementation #1341 - fix js implementation #1343 - fix js implementation
#1039 - Don't change implementation, clarify the SNIP
#1342 - Don't change implementation, clarify the snip, have a new revision if the change is necessary #1348 - Don't change implementation, clarify the snip, have a new revision if the change is necessary
@sgc-code, @penovicp @xJonathanLEI @tabaktoni @franciszekjob
Thanks @amanusk. Please let us know when these changes are released so I can update Starknet.go code
#1039 - Don't change implementation, clarify the SNIP #1342 - Don't change implementation, clarify the snip, have a new revision if the change is necessary #1348 - Don't change implementation, clarify the snip, have a new revision if the change is necessary
This were actually addressed in this PR
I think it's missing the fixes for these two issues: https://github.com/starknet-io/starknet.js/issues/1287 and https://github.com/starknet-io/starknet.js/issues/1362
#1039 - Don't change implementation, clarify the SNIP #1342 - Don't change implementation, clarify the snip, have a new revision if the change is necessary #1348 - Don't change implementation, clarify the snip, have a new revision if the change is necessary
This were actually addressed in this PR
Like the previous aggregate, the idea here is to have them all available in one place, from there to evaluate potential impact and pick which will be applied and in what way.
To address the other message here:
- #1287 - It was picked out in a similar approach as previously mentioned and has been included since v6.24.1.
- #1362 - I addressed in a comment there. The SNIP doesn't foresee the usage of the requested type/message format, and a more verbose one should be used.