Skip to content

Conversation

@lebojo
Copy link
Member

@lebojo lebojo commented Dec 15, 2025

This pull request adds Matomo analytics tracking to all message banner actions in the Mail app, enabling better monitoring of user interactions with banners.
The changes introduce a new matomoName property to the MessageBanner enum for consistent event naming, and ensure that analytics events are sent when users interact with these banners.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds Matomo analytics tracking to message banner interactions in the Infomaniak Mail iOS app. The changes introduce a centralized approach for tracking user interactions with various message banners (spam, display content, encryption, unsubscribe, acknowledgement, and schedule).

Key Changes:

  • Added a new messageBanner event category to Matomo tracking
  • Introduced a matomoName computed property to the MessageBanner enum for consistent event naming
  • Implemented analytics tracking across all message banner user interactions

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
MailCore/Utils/Matomo+Extension.swift Adds new messageBanner event category for analytics
MailCore/Models/MessageBanner.swift Adds matomoName computed property returning analytics event names for each banner type
Mail/Views/Thread/Message/MessageScheduleHeaderView.swift Tracks analytics when user changes schedule date
Mail/Views/Thread/Message/MessageEncryptionHeaderView.swift Tracks analytics when user views concerned recipients for encrypted messages
Mail/Views/Thread/Message/MessageBannerHeaderView.swift Adds helper method and tracking for display content, spam, unsubscribe, and acknowledge banner actions

Copy link
Member

@valentinperignon valentinperignon left a comment

Choose a reason for hiding this comment

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

Just a few comments, great work

@lebojo lebojo force-pushed the fix/add-matomo-for-message-banner branch from 5fb6ad5 to 25d24ee Compare December 16, 2025 09:26
@lebojo lebojo force-pushed the fix/add-matomo-for-message-banner branch from 25d24ee to 323814c Compare December 16, 2025 09:30
@sonarqubecloud
Copy link

@lebojo lebojo merged commit 52d964a into master Dec 16, 2025
11 checks passed
@lebojo lebojo deleted the fix/add-matomo-for-message-banner branch December 16, 2025 09:49
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