Project

General

Profile

Cleanup #46750

mgr/dashboard: Merge disable and disableDesc table action methods

Added by Tiago Melo about 3 years ago. Updated over 2 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
General
Target version:
% Done:

0%

Tags:
Backport:
octopus
Reviewed:
Affected Versions:
Pull request ID:

Description

Currently we have to separate methods to calculate if a action is disabled and if it should display any description.
By merging both we reduce duplicated code, keeping the same functionality.


Related issues

Copied to Dashboard - Backport #47943: octopus: mgr/dashboard: Merge disable and disableDesc table action methods Resolved
Copied to Dashboard - Backport #49624: nautilus: mgr/dashboard: Merge disable and disableDesc table action methods Rejected

History

#1 Updated by Tiago Melo about 3 years ago

  • Status changed from In Progress to Fix Under Review
  • Pull request ID set to 36345

#2 Updated by Lenz Grimmer about 3 years ago

  • Tags set to low-hanging-fruit, refactoring
  • Target version set to v16.0.0

#3 Updated by Tiago Melo about 3 years ago

  • Status changed from Fix Under Review to Resolved

#4 Updated by Kiefer Chang almost 3 years ago

  • Backport set to octopus

Add octopus backport because this is needed for #47019

#5 Updated by Kiefer Chang almost 3 years ago

  • Status changed from Resolved to Pending Backport

#6 Updated by Nathan Cutler almost 3 years ago

  • Copied to Backport #47943: octopus: mgr/dashboard: Merge disable and disableDesc table action methods added

#7 Updated by Nathan Cutler almost 3 years ago

  • Status changed from Pending Backport to Resolved

While running with --resolve-parent, the script "backport-create-issue" noticed that all backports of this issue are in status "Resolved" or "Rejected".

#8 Updated by Ernesto Puerta over 2 years ago

  • Status changed from Resolved to Pending Backport
  • Backport changed from octopus to octopus, nautilus

#9 Updated by Backport Bot over 2 years ago

  • Copied to Backport #49624: nautilus: mgr/dashboard: Merge disable and disableDesc table action methods added

#10 Updated by Ernesto Puerta over 2 years ago

  • Project changed from mgr to Dashboard
  • Category changed from 132 to General

#11 Updated by Ernesto Puerta over 2 years ago

  • Status changed from Pending Backport to Resolved
  • Backport changed from octopus, nautilus to octopus

Also available in: Atom PDF