Skip to content

Conversation

@mcleinman
Copy link
Collaborator

@mcleinman mcleinman commented Nov 5, 2025

Description

Moving towards giving more descriptive update and what's new messages. It's possible some of these features won't make it, and we'll hold the bullet point for the following release.

Reference

VPN-7324

Checklist

  • My code follows the style guidelines for this project
  • I have not added any packages that contain high risk or unknown licenses (GPL, LGPL, MPL, etc. consult with DevOps if in question)
  • I have performed a self review of my own code
  • I have commented my code PARTICULARLY in hard to understand areas
  • I have added thorough tests where needed

@mcleinman mcleinman requested a review from oskirby November 5, 2025 23:26
@mcleinman mcleinman requested a review from flodolo as a code owner November 5, 2025 23:26
Copy link
Collaborator

@oskirby oskirby left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@flodolo flodolo left a comment

Choose a reason for hiding this comment

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

A couple of nits/questions. I can't say that I'm a fan of this style of messaging for release notes.

What is the timing of the v2.33 release? How much time do we have for localization? That's an unexpected amount of content for this type of message.


233updateMessage:
bullet1:
value: "[all] Finish transitioning subscription management to the Mozilla Accounts website."
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
value: "[all] Finish transitioning subscription management to the Mozilla Accounts website."
value: "[all] Finish transitioning subscription management to the Mozilla accounts website."

Did you mean "Finished"? "Finish" sounds like a plan, not a release note. Also, accounts always lowercase.

value: "[all] Finish transitioning subscription management to the Mozilla Accounts website."
comment: Bullet point with a specific update in 2.33. Should have tag that indicates it applies to all platforms.
bullet2:
value: "[all] Added regular key rotation, improving security in the event of extreme, unexpected situations."
Copy link
Collaborator

Choose a reason for hiding this comment

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

extreme, unexpected situations sounds unnecessarily ominous. What are those?

value: "[Windows, macOS, Linux] In a rare situation, the status icon wasn’t updating. We dug in deep, and it should again work for everyone."
comment: Bullet point with a specific update in 2.33. Should have tag that indicates it applies to the 3 desktop platforms.
bullet6:
value: "[Linux] If your distro ships with that latest Qt 6.10, builds from scratch may have had issues. They no longer should."
Copy link
Collaborator

Choose a reason for hiding this comment

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

This seems relevant for an extremely small number of users? Will they even see the update message if they build their own?

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