Closed Bug 1928399 Opened 1 year ago Closed 1 year ago

Release channel should not open whatnew page

Categories

(Thunderbird :: Build Config, defect, P2)

Thunderbird 132
defect

Tracking

(thunderbird132 affected, thunderbird133+ fixed)

RESOLVED FIXED
134 Branch
Tracking Status
thunderbird132 --- affected
thunderbird133 + fixed

People

(Reporter: corey, Assigned: rjl)

Details

Attachments

(1 file)

The Release channel shouldn't open a whatnew page.
See more at original issue at https://github.com/thunderbird/thunderbird-website/issues/683

By the way, it looks like we have a whatsnew url defined for beta as well in mail/config/whats_new_page.yml. Is that needed?

Remove lines 27-35 of that file.
Though once we have release notes for monthly, we might want it back? Comment it out maybe?

Actually, there's another way to do this that I think makes more sense.

in /comm/taskcluster/kinds/balrog/kind.yml, there's:

        update-no-wnp:
            by-release-type:
                nightly.*: false
                default: true

Let's change that so that the "release" release type is set to "false" like "nightly" is. That will change the Balrog update rules to point to the "-No-WNP" release blobs. The only difference with those is that the "updateLine" section will not include the whatsnew page in the showURL action.

"updateLine": [
    {
      "fields": {
        "detailsURL": "https://live.thunderbird.net/thunderbird/releasenotes?locale=%LOCALE%&version=132.0&channel=release",
        "type": "minor"
      },
      "for": {}
    },
    {
      "fields": {
        "actions": "showURL",
        "openURL": "https://live.thunderbird.net/thunderbird/whatsnew?locale=%LOCALE%&version=132.0&channel=release&oldversion=%OLD_VERSION%"
      },
      "for": {
        "channels": [
          "release",
          "release-localtest",
          "release-cdntest"
        ],
        "versions": [
          "<132.0"
        ]
      }
    }
  ]

This change can be made manually on the release channel now and fix this issue for current users. The code change is needed to make it permanent.

Assignee: nobody → rob
Status: NEW → ASSIGNED
Target Milestone: --- → 134 Branch

Comment on attachment 9435205 [details]
Bug 1928399 - Use -no-WNP release blobs on comm-release. r=#thunderbuild

[Approval Request Comment]
Regression caused by (bug #): N/A
User impact if declined: will keep opening a 404'd url on major update
Testing completed (on c-c, etc.):
Risk to taking this patch (and alternatives if risky):
as requested. low risk, this is a minor configuration change to a code path used by nightly and FF devedition.

Attachment #9435205 - Flags: approval-comm-beta?

Pushed by arschmitz@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/82f9dc077e4f
Use -no-WNP release blobs on comm-release. r=dandarnell

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED

Comment on attachment 9435205 [details]
Bug 1928399 - Use -no-WNP release blobs on comm-release. r=#thunderbuild

[Triage Comment]
Approved for beta

Attachment #9435205 - Flags: approval-comm-beta? → approval-comm-beta+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: