2day icon indicating copy to clipboard operation
2day copied to clipboard

Task Frequency Picker doesn't display the correct value on edit

Open webOSpinn opened this issue 8 years ago • 2 comments

The Task Frequency Picker displays the last entered value for the "Every..." option instead of the saved value on edit.

To reproduce this issue: Enter a task with a frequency of "every 2 weeks" and save it. wp_ss_20180204_0001

Then save a task with a frequency of "every 2 days" and save it. wp_ss_20180204_0004

Open the first task and edit the frequency. Instead of "every 2 weeks" it still says "every 2 days". wp_ss_20180204_0005

webOSpinn avatar Feb 05 '18 04:02 webOSpinn

I'm not able to repro this one. Could it be fixed by your other PR that was already merged?

japf avatar Feb 07 '18 18:02 japf

Interesting. I double checked and I am able to consistently recreate the issue on desktop and mobile with the current store release and with a build containing my pull requests. I don't believe that any of my changes would have fixed this.

I wonder what I'm doing differently that is causing the frequency picker to display the last entered value instead of the value saved in the task?

This issue is easiest to reproduce with the "Every..." option but I can get this issue to happen when I select "Every day", "Every week", "Every month", "Every year". To do this:

  1. Select "Every day"
  2. Now select the frequency again. It should say "each day(s)" (going back to view the frequency is key to making this happen)
  3. Cancel out of this screen.
  4. Save the task
  5. Now open another task that already has a recurrence of "Every..."
  6. Edit the frequency and it will still say "each day(s)" instead of the actual recurrence on the task.

I don't see this same behavior when I select "Specific week days..." or "Specific month day..."

webOSpinn avatar Feb 08 '18 05:02 webOSpinn