Skip to content

release: join a study AJ-650 - #263

Merged
amitsinghsutara merged 31 commits into
release-AJ-650-join_a_studyfrom
develop
Jun 2, 2026
Merged

release: join a study AJ-650#263
amitsinghsutara merged 31 commits into
release-AJ-650-join_a_studyfrom
develop

Conversation

@dz4va

@dz4va dz4va commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

Changes

  • Preserve the original cr_user_id while storing study_user_id separately.
  • Make study enrollment write-once so later app links do not reopen the modal or overwrite the saved phone-number ID.
  • Fix Play Store deferred deeplink parsing so language, source, and campaign ID are extracted independently.

How to test

  • Production release

Ref: AJ-650

Rajesh1041 and others added 30 commits May 10, 2026 10:57
MR-72 Update DefaultAppEventPayloadHandler.java
fix: [MR-84] added container version to firestore payload
Add ID confirmation dialog UI and improve project configuration
- Persist confirmed study_user_id after successful study enrollment
- Store the study ID alongside the existing pseudoId using a dedicated preference key
- Append study_user_id only to app_launch Firebase event parameters when available
- Leave unenrolled app_launch events unchanged by omitting study_user_id
- Add unit coverage for app_launch events with and without a persisted study_user_id
- Restore the existing language_selected analytics test invocation so the test class passes
feat: add study user id to app launch analytics
feat: replace toast with custom success dialog
refactor:  Disabling cr_user_id modification!
…nk language handling

- Preserve the original cr_user_id while storing study_user_id separately.
- Make study enrollment write-once so later app links do not reopen the modal or overwrite the saved phone-number ID.
- Fix Play Store deferred deeplink parsing so language, source, and campaign ID are extracted independently.
@dz4va
dz4va requested a review from amitsinghsutara June 2, 2026 14:30
@dz4va dz4va self-assigned this Jun 2, 2026
@coderabbitai

coderabbitai Bot commented Jun 2, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (3)
  • develop
  • test
  • main

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 65ae2324-b792-43f8-8738-5a574adbea3c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch develop

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@amitsinghsutara
amitsinghsutara merged commit b1a22ff into release-AJ-650-join_a_study Jun 2, 2026
3 checks passed
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.

4 participants