Page MenuHomePhorge

Allow for reordering milestones
Needs RevisionPublic

Authored by Cigaryno on Apr 5 2023, 17:23.
Referenced Files
Unknown Object (File)
Thu, Apr 11, 05:10
Unknown Object (File)
Wed, Apr 10, 19:29
Unknown Object (File)
Sun, Apr 7, 04:32
Unknown Object (File)
Mon, Apr 1, 01:09
Unknown Object (File)
Sun, Mar 31, 14:47
Unknown Object (File)
Mon, Mar 25, 19:36
Unknown Object (File)
Thu, Mar 21, 01:57
Unknown Object (File)
Feb 24 2024, 05:00

Details

Summary
This revision's diff has been copied from Wikimedia. Please warn me if there is an error.

Ref T15082

Test Plan

Reorder milestones in the Subprojects menu.

Diff Detail

Repository
rP Phorge
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Cigaryno requested review of this revision.Apr 5 2023, 17:23
Cigaryno edited the test plan for this revision. (Show Details)

Maybe I can propose to control this new behavior by a new method:

public isMilestoneReordable() {
   // historically this was false, but hey, this is a community edition
   return true;
}

So that Evan can keep this to false
So other installations can customize this thing

Since this change is highly controversial from the opinion of the glorious Evan, I would like to propose to introduce a flag, like this one:

public isMilestoneReordable() {
   return true;  // if we are bold
   return false; // if we are not bold
}

Premising I have NOT a strong opinion about the default value, I think that a dedicated activation method is strongly necessary to allow Evan to disable this more easily, or Wikimedia to enable this more easily, without very custom ad-hoc code.

This revision now requires changes to proceed.May 3 2023, 09:50