HomePhabricator
Diffusion Comm 5aed2ee33018

[identity] Store social proof as a Map in DDB

Description

[identity] Store social proof as a Map in DDB

Summary:
Instead of JSON-serializing and deserializing social proof, we can store it directly as a map attribute in DDB.

NOTE: Note this is breaking change - previous users with String social proof in DDB will fail. I found only 3 wallet users on staging and none on prod. I prefer manually migrating (or deleting) these 3 rows instead of keeping this code backwards compatible

Depends on D11547

Test Plan:

  • Added unit test
  • Registered a wallet user and called FindUserIdentity to make sure that social proof is retrieved

Reviewers: varun, will

Reviewed By: varun

Subscribers: ashoat, tomek

Differential Revision: https://phab.comm.dev/D11548

Details

Provenance
bartekAuthored on Thu, Apr 4, 4:04 AM
Reviewer
varun
Differential Revision
D11548: [identity] Store social proof as a Map in DDB
Parents
rCOMM883fffc6adc1: [protos] Refactor user Identity message type
Branches
Unknown
Tags
Unknown