Skip to content

MS-1062 Add logging to application class to attempt to track the state#1301

Merged
luhmirin-s merged 1 commit into
mainfrom
feature/MS-1062-app-lifecycle-analytics
Aug 4, 2025
Merged

MS-1062 Add logging to application class to attempt to track the state#1301
luhmirin-s merged 1 commit into
mainfrom
feature/MS-1062-app-lifecycle-analytics

Conversation

@luhmirin-s
Copy link
Copy Markdown
Contributor

JIRA ticket
Will be released in: 2025.3.0

Notable changes

  • Add logging to the application lifecycle methods.

Additional work checklist

  • Effect on other features and security has been considered
  • Design document marked as "In development" (if applicable)
  • External (Gitbook) and internal (Confluence) Documentation is up to date (or ticket created)
  • Test cases in Testiny are up to date (or ticket created)
  • Other teams notified about the changes (if applicable)

@luhmirin-s luhmirin-s requested review from a team, BurningAXE, TristramN, alex-vt, alexandr-simprints, meladRaouf and ybourgery and removed request for a team August 4, 2025 08:34
@cla-bot cla-bot Bot added the ... label Aug 4, 2025
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Aug 4, 2025


override fun onCreate() {
super.onCreate()
Simber.i("Application created", tag = APPLICATION)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Could this make the logs too noisy?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Once per process creation should be fine.

@luhmirin-s luhmirin-s merged commit 2920255 into main Aug 4, 2025
13 checks passed
@luhmirin-s luhmirin-s deleted the feature/MS-1062-app-lifecycle-analytics branch August 4, 2025 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants