Read the issue and seems like it was unrelated to this code.

• Made get_user_identifier() return Option
• Updated callsites to return Status::not_found if missing identifier

Requesting changes to discuss possible usage of BatchGetItem

In D11412#329178, @ashoat wrote:

@varun looking at the test plan here, is it fair to say you weren't able to reproduce ENG-7390? Is it worth trying to test via the UI to see if that reproduces it?

In D11410#329193, @ashoat wrote:

Do we need to update anything for registerPasswordUser or registerWalletUser on web?

Here's the linear issue for replacing usages of the FIDProvider and reverting once it's no longer needed:
https://linear.app/comm/issue/ENG-7590/replace-all-fidprovider-usages-with-synced-metadata-store

Guessing the plumbing for native (and the rest of the plumbing for web) will come in a later diff

Do we need to update anything for registerPasswordUser or registerWalletUser on web?

@varun looking at the test plan here, is it fair to say you weren't able to reproduce ENG-7390? Is it worth trying to test via the UI to see if that reproduces it?

add support for the remove synced metadata action type

rebase, but nothing changed

remove TODOs. Here's the linear task, instead: https://linear.app/comm/issue/ENG-7588/replace-params-in-native-rust-library-methods-with-serialized-structs