[MOB-10091]: AUT Pre-Bash Updates #475
Conversation
| @@ -62,7 +66,7 @@ export const LoginFormWithoutJWT: FC<Props> = ({ | |||
|
|
|||
| const setUser = useEmail ? setEmail : setUserId; | |||
|
|
|||
| setUser(user, true) | |||
| setUser(user) | |||
There was a problem hiding this comment.
I'm still unsure how identityResolution is being used, should we expect it to be passed in?
There was a problem hiding this comment.
Here, identity resolution could be passed in. The config defaults are set to mergeOnAnonymousToKnown: true so just removing the variable here because it's redundant. The values for IdentityResolution, i.e., mergeOnAnonymousToKnown and replayOnVisitorToKnown could be passed in and changed any time setEmail or setUserId is called.
There was a problem hiding this comment.
Ah so it's an override and there's logic in the implementation that pulls from the config? If so, could we label it as something like identityResolutionOverride in the function signatures where it would be an override?
JIRA Ticket(s) if any
Description
Renames config var, cleans up sample app without JWT fixes
Test Steps