Page MenuHomePhabricator
Differential D10764

[identity] Store social proof in users table
ClosedPublic
Actions

Authored by bartek on Jan 22 2024, 3:06 AM.
Tags
None
Referenced Files
F3156377: D10764.diff
Tue, Nov 5, 3:16 PM
Unknown Object (File)
Fri, Oct 18, 3:23 AM
Unknown Object (File)
Fri, Oct 18, 3:23 AM
Unknown Object (File)
Fri, Oct 18, 3:23 AM
Unknown Object (File)
Fri, Oct 18, 3:23 AM
Unknown Object (File)
Sep 21 2024, 6:07 PM
Unknown Object (File)
Sep 6 2024, 9:50 PM
Unknown Object (File)
Sep 6 2024, 9:50 PM
View All 48 Files
Subscribers
ashoat
tomek

Details

Reviewers
varun
michal
will
Commits
rCOMMb172225d3ce4: [identity] Store social proof in users table
Summary

Part 1 of ENG-6480.
When wallet user is being registered, we should store the social proof in the users table.

Depends on D10762

Test Plan

Called LogInWalletUser RPC and checked (awslocal dynamodb scan) that the social proof is stored in the users table.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

bartek created this revision.Jan 22 2024, 3:06 AM
bartek held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptJan 22 2024, 3:06 AM
bartek added a child revision: D10765: [identity] Remove social proof from devices table.Jan 22 2024, 3:08 AM
Harbormaster completed remote builds in B25987: Diff 35919.Jan 22 2024, 3:43 AM
bartek published this revision for review.Jan 22 2024, 3:47 AM
bartek added inline comments.
services/identity/src/constants.rs
51–52

previous name was misleading, especially since we no longer have the "devices" map attribute

varun accepted this revision.Jan 22 2024, 6:04 PM
This revision is now accepted and ready to land.Jan 22 2024, 6:04 PM
Closed by commit rCOMMb172225d3ce4: [identity] Store social proof in users table (authored by bartek). · Explain WhyJan 26 2024, 8:33 AM
This revision was automatically updated to reflect the committed changes.
bartek added a commit: rCOMMb172225d3ce4: [identity] Store social proof in users table.

Revision Contents
Changeset List

PathSize
services/
identity/
src/
3 lines
24 lines
6 lines

Diff 35919

View Options

services/identity/src/constants.rs

Loading...
View Options

services/identity/src/database.rs

Loading...
View Options

services/identity/src/ddb_utils.rs

Loading...