Skip to content

analytics: Update labels for mobile apps for Flutter launch. #34849

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

Merged
merged 1 commit into from
Jun 14, 2025

Conversation

laurynmm
Copy link
Collaborator

@laurynmm laurynmm commented Jun 9, 2025

Updates the labels in the messages by client chart so that the Flutter app is no longer labeled as beta and the React Native app is now labeled as old. The older Android and iOS apps are now labeled as ancient.

Part of: zulip/zulip-flutter#715.
Relevant CZO discussion: #api design > zulip-flutter in analytics client chart @ 💬


Screenshots and screen captures:

from Analytics realm in dev environment
Screenshot from 2025-06-11 10-57-27


Self-review checklist
  • Self-reviewed the changes for clarity and maintainability
    (variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

  • Explains differences from previous plans (e.g., issue description).
  • Highlights technical choices and bugs encountered.
  • Calls out remaining decisions and concerns.
  • Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

  • Each commit is a coherent idea.
  • Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

  • Visual appearance of the changes.
  • Responsiveness and internationalization.
  • Strings and tooltips.
  • End-to-end functionality of buttons, interactions and flows.
  • Corner cases, error conditions, and easily imagined bugs.

@zulipbot
Copy link
Member

zulipbot commented Jun 9, 2025

Hello @zulip/server-analytics members, this pull request was labeled with the "area: analytics" label, so you may want to check it out!

@laurynmm
Copy link
Collaborator Author

laurynmm commented Jun 9, 2025

CI failure installing dependencies is not related to changes in this PR.

Updates the labels in the"Messages set by client" chart so that
the Flutter app is no longer labeled as "beta".

The React Native app is now labeled as "old". And the older
Android and iOS apps are now labeled as "ancient".
@laurynmm laurynmm force-pushed the analytics-relabel-mobile-apps branch from 34ae319 to 032f2ad Compare June 11, 2025 08:58
@laurynmm
Copy link
Collaborator Author

laurynmm commented Jun 11, 2025

Updates:

  • The new label for the React Native app is "Old mobile app (React Native)"
  • The new label for the older mobile apps are "Ancient Android app" and "Ancient iOS app"

Screenshot has been updated in main PR comment for these updates.

@timabbott timabbott merged commit 3aadb86 into zulip:main Jun 14, 2025
7 checks passed
@timabbott
Copy link
Member

Meged, thanks @laurynmm!

@gnprice gnprice mentioned this pull request Jun 14, 2025
2 tasks
@laurynmm laurynmm deleted the analytics-relabel-mobile-apps branch June 16, 2025 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants