No Code Attached Yet
avatar PhilETaylor
PhilETaylor
27 Dec 2021

Steps to reproduce the issue

So lets say my task has been constantly failing and I have 3123 Times Failed. I have fixed the problem and want to reset the stats to that I can keep an eye on it, there is no way to do that without deleting the task and creating a new one.

Screenshot 2021-12-27 at 23 26 04

@ditsuke

avatar PhilETaylor PhilETaylor - open - 27 Dec 2021
avatar joomla-cms-bot joomla-cms-bot - change - 27 Dec 2021
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 27 Dec 2021
avatar PhilETaylor PhilETaylor - change - 27 Dec 2021
The description was changed
avatar PhilETaylor PhilETaylor - edited - 27 Dec 2021
avatar pritam825
pritam825 - comment - 26 Jan 2022

Screenshot (1962)
is this the correct position for the reset button?

avatar ditsuke
ditsuke - comment - 30 Jan 2022

Screenshot (1962) is this the correct position for the reset button?

@pritam825 it should go under "Execution History". Also I think the button should be inlined into a field (for example the calendar and user buttons besides "Created" an "Created By".

avatar pritam825
pritam825 - comment - 30 Jan 2022

@ditsuke is this fine?
Screenshot (1976)

avatar pritam825
pritam825 - comment - 30 Jan 2022

or you are taking about this, here in the input field there will two values, one is yes and another one is no, and default value will be no, when we will click on yes, then it will so a warning message and after clicking "ok" on the warning button, it resets the history
Screenshot (1979)

avatar ditsuke
ditsuke - comment - 30 Jan 2022

@pritam825 take a look at this reset button for user API tokens. I believe we should have a similar radio here. You can have a look at the underlying form XML at plugins/user/token/forms/token.xml

image

avatar Quy
Quy - comment - 30 Jan 2022

Or add a reset button next to Time Executed and Times Failed. See Banners component where this is done.

avatar brianteeman
brianteeman - comment - 30 Jan 2022

Definitely this way is the way to go

image

avatar ditsuke
ditsuke - comment - 30 Jan 2022

Definitely this way is the way to go

image

Better yet! Inline these buttons for supported stats. Although not sure if Times Failed and Times Executed should be decoupled.

avatar brianteeman
brianteeman - comment - 4 Mar 2022

please test #37201

avatar pritam825
pritam825 - comment - 4 Mar 2022

@brianteeman , I had also done the same thing, by getting and understanding the code of banners, but You created the PR first

avatar pritam825
pritam825 - comment - 4 Mar 2022

Can I create mine?

avatar brianteeman
brianteeman - comment - 4 Mar 2022

if its different sure.

avatar pritam825
pritam825 - comment - 4 Mar 2022

It's almost same,the only change in the file name, I have saved the file name with Times_executed_field and Times_failled_field ?, and the code is same because the same logic as of ClickField.php in com_banners

avatar pritam825
pritam825 - comment - 4 Mar 2022

so, can I create mine? ?

avatar richard67
richard67 - comment - 4 Mar 2022

It seems Brian was faster. That can happen sometimes.

avatar richard67 richard67 - close - 4 Mar 2022
avatar richard67
richard67 - comment - 4 Mar 2022

Closing as having a pull request. Please test #37201 . Thanks in advance.

avatar richard67 richard67 - change - 4 Mar 2022
Status New Closed
Closed_Date 0000-00-00 00:00:00 2022-03-04 13:16:56
Closed_By richard67
avatar pritam825
pritam825 - comment - 4 Mar 2022

@richard67 yes, Thanks

avatar richard67 richard67 - change - 25 May 2022
Status Closed New
Closed_Date 2022-03-04 13:16:56
Closed_By richard67
avatar richard67 richard67 - reopen - 25 May 2022
avatar richard67
richard67 - comment - 25 May 2022

Re-opening since the pull request has been closed by it’s author.

avatar PhilETaylor PhilETaylor - change - 17 Aug 2022
Status New Closed
Closed_Date 0000-00-00 00:00:00 2022-08-17 08:53:16
Closed_By PhilETaylor
avatar PhilETaylor
PhilETaylor - comment - 17 Aug 2022
avatar PhilETaylor PhilETaylor - close - 17 Aug 2022

Add a Comment

Login with GitHub to post a comment