Consider making `metadata.parentId` optional

TBD54566975 / tbdex

56 stars 25 forks source link

Consider making `metadata.parentId` optional #138

Closed mistermoe closed 1 year ago

mistermoe commented 1 year ago

making metadata.parentId a required field necessitates that Alice stores an entire tbDEX thread on her side as well which seems unnecessarily encumbering especially in scenarios where Alice does not have the latest thread state despite assuming the contrary (e.g. Alice goes to submit an Order but the PFI has Closed)

props to @jiyoontbd for calling this out

mistermoe commented 1 year ago

Related comment by @KendallWeihe: https://github.com/TBD54566975/tbdex-protocol/pull/137#discussion_r1300765595

mistermoe commented 1 year ago

removed parentId for now