Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AUT Feature #404

Open
wants to merge 106 commits into
base: main
Choose a base branch
from
Open

AUT Feature #404

wants to merge 106 commits into from

Conversation

hardikmashru
Copy link
Collaborator

@hardikmashru hardikmashru commented Jun 7, 2024

JIRA Ticket(s) if any

Description

Test Steps

hani-iterable and others added 24 commits March 4, 2024 09:07
* merge fork repo code and implement anon session

* resolve test cases failure

* call API for getting criteria

* track anon event and purchase

* track anon update cart and user

* write test cases

* resolve test cases failure for autjorization

* modify anonymous manager class

* modify test cases

* update the lock files

* implement criteria completion checker

* implement user merge and set config

* set config

* Complete AUT feature

* Resolve review comments

* implement test case for merge user

* unit test case for anon user event tracking

* test case for criteria completion checker

* resolve build failure

* resolve build failure

* resolve review comments

* remove circular dependency

* remove lock file

* resolve review comments
@mprew97
Copy link
Contributor

mprew97 commented Jun 10, 2024

@hardikmashru what's the difference between this and the AUT branch? Are there new additions? Hard to tell what to review.

Also, the build appears to be failing so we ought to address that.

@hardikmashru
Copy link
Collaborator Author

@hardikmashru what's the difference between this and the AUT branch? Are there new additions? Hard to tell what to review.

Also, the build appears to be failing so we ought to address that.

Hi @mprew97 Please do not review other branch AUT, please review this one only.

Copy link
Contributor

@mprew97 mprew97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we need to base this off of our Embedded GA work because there are quite a lot of architectural shifts and features that this branch should be making use of.

For example, the updates to network interceptors, the ability in the sample app to toggle between userId and email, and many others.

src/utils/anonymousUserMerge.test.ts Outdated Show resolved Hide resolved
src/utils/anonymousUserEventManager.ts Outdated Show resolved Hide resolved
src/users/users.ts Outdated Show resolved Hide resolved
src/users/users.schema.ts Outdated Show resolved Hide resolved
src/users/types.ts Outdated Show resolved Hide resolved
react-example/src/views/Commerce.tsx Outdated Show resolved Hide resolved
react-example/src/views/Commerce.tsx Outdated Show resolved Hide resolved
react-example/src/index.tsx Outdated Show resolved Hide resolved
react-example/src/components/LoginForm.tsx Outdated Show resolved Hide resolved
react-example/src/components/LoginForm.tsx Outdated Show resolved Hide resolved
darshan-iterable and others added 20 commits September 17, 2024 08:43
* [MOB-9505] rename merge parameter

* fix ci failure and resolved comment

* fix: ci failure and merged AUT_main changes
* [MOB-9402] update user should not be a separate call

* feat: test cases update user should not be a separate call
* MOB-9650 Added support for nested criteria match a.b.c

* fix: removed updatecart from nested criteria
* MOB-9650 Added support for nested criteria match a.b.c

* fix: removed updatecart from nested criteria

* [MOB-9652] support for nested JSON array

* [MOB-9652] customEvent test case for nested JSON array

* [MOB-9168] Automated unit tests against complex criteria (#461)

* [MOB-9168] Automated unit tests against complex criteria

* [MOB-9168] Automated unit tests against complex criteria
…d anon userid (#460)

* [MOB-9578] implements identity resolution

* [MOB-9639] Added handler for notifying customer app of a newly created anon userid

* moved onAnonUserCreated in identityResolution
* [MOB-9640] Keep AUT off until concent to track is granted

* rename concent to consent

* fixed eslint issue

* added consent support for with-jwt config
* AUT bug bash settings

* block api calls if typeOfAuth not set

* auth checks and tests

* branch fix reversions

* fix most of tests

---------

Co-authored-by: jyu115 <[email protected]>
…ated in Config (#464)

* shuffle onAnonUserCreated

* allow identity resolution overrides on setEmail/setUserId
* fix replay issue with JWT

* fix spec
…alls (#467)

* fix replay issue with JWT

* fix spec

* move initialize check over

* add type of auth util

* clean up auth checks and circular deps

* add getter

* fix

* lets see

* add additional endpoints
…468)

* prettify code

* [MOB-9703] Added support for fetching new JWT prior to calling merge

* moved JWT to localstorage

* fix circular deps
* [MOB-9970] anonymous criteria should match for nested values

* [MOB-10064] single primitive array bug fix
* update var, some error cleanup in sample app

* remove endpoint from list

* fix spec
* clear anon data on user initialization

* clear anon data even if replay is false

* update tests

* oop

* Update authorization.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants