Skip to content

Conversation

@shubham1g5
Copy link
Contributor

Product Description

https://dimagi.atlassian.net/browse/CCCT-1472

Labels and Review

  • Do we need to enhance the manual QA test coverage ? If yes, the "QA Note" label is set correctly
  • Does the PR introduce any major changes worth communicating ? If yes, the "Release Note" label is set and a "Release Note" is specified in PR description.
  • Risk label is set correctly
  • The set of people pinged as reviewers is appropriate for the level of risk of the change

analyticsInstance.setUserProperty(CCAnalyticsParam.CCC_ENABLED,
String.valueOf(PersonalIdManager.getInstance().isloggedIn()));

analyticsInstance.setUserProperty(CCAnalyticsParam.BUILD_NUMBER, String.valueOf(BuildConfig.VERSION_CODE));
Copy link
Contributor

Choose a reason for hiding this comment

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

@shubham1g5 BuildConfig.VERSION_CODE is showing 1 for master branch, does it changes when creating production build?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

that's right, happens with this method

@shubham1g5 shubham1g5 merged commit 2197f68 into Beta_2.58_474459 Jul 18, 2025
1 check passed
@shubham1g5 shubham1g5 deleted the addBuildNumberToAnalytics branch July 18, 2025 07:38
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.

3 participants