Project

General

Profile

Actions
Copy link

Task #8595

closed
SB SB

flowbits: remove "toggle" command

Task #8595: flowbits: remove "toggle" command

Added by Shivani Bhardwaj about 1 month ago. Updated 11 days ago.

Status:
Closed
Priority:
Normal
Assignee:
Shivani Bhardwaj
Target version:
9.0.0-beta1
Effort:
Difficulty:
low
Label:

Description

flowbits toggle command is not used by any major ruleset provider.
All the functionality that toggle offers can already be achieved by the combination of other available commands that are in regular use.
State tracking for flowbits becomes a bit simplified if one of the commands is removed.

Subtasks 2 (0 open — 2 closed)

Task #8638: flowbits: deprecate "toggle" commandClosedShivani BhardwajActions
Task #8639: flowbits: deprecate "toggle" commandClosedShivani BhardwajActions

SB Updated by Shivani Bhardwaj about 1 month ago Actions
Copy link
 #1

  • Subject changed from flowbits: deprecated "toggle" command to flowbits: deprecate "toggle" command

SB Updated by Shivani Bhardwaj about 1 month ago Actions
Copy link
 #2

  • Difficulty set to low

SB Updated by Shivani Bhardwaj about 1 month ago Actions
Copy link
 #3

  • Status changed from Assigned to In Review

PA Updated by Philippe Antoine about 1 month ago Actions
Copy link
 #4

All the functionality that toggle offers can already be achieved by the combination of other available commands that are in regular use.

I am not sure it is the case. How do you handle this ?

I am not in favor of deprecating this feature directly.

State tracking for flowbits becomes a bit simplified if one of the commands is removed.

We could just print a warning when this command is used and not do the state tracking...

SB Updated by Shivani Bhardwaj 28 days ago Actions
Copy link
 #5

  • Status changed from In Review to Closed

VJ Updated by Victor Julien 28 days ago Actions
Copy link
 #6

How is adding the notices to 7 and 8 tracked?

SB Updated by Shivani Bhardwaj 26 days ago Actions
Copy link
 #7

I'll create a tracking ticket for that. Realizing this ticket's title is wrong. 8 and 7 should use "deprecated" and this one should be about the removal altogether.

SB Updated by Shivani Bhardwaj 26 days ago Actions
Copy link
 #8

  • Subject changed from flowbits: deprecate "toggle" command to flowbits: remove "toggle" command

SB Updated by Shivani Bhardwaj 26 days ago Actions
Copy link
 #9

  • Status changed from Closed to Resolved

SB Updated by Shivani Bhardwaj 26 days ago Actions
Copy link
 #10

  • Subtask #8638 added

SB Updated by Shivani Bhardwaj 26 days ago Actions
Copy link
 #11

  • Subtask #8639 added

PA Updated by Philippe Antoine 11 days ago Actions
Copy link
 #12

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: PDF Atom