cwa-documentation icon indicating copy to clipboard operation
cwa-documentation copied to clipboard

Contact Journal: Cannot set place duration to 24h

Open MikeMcC399 opened this issue 3 years ago • 7 comments

Avoid duplicates

  • [X] Bug is not mentioned in the FAQ
  • [X] Bug is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
  • [X] Bug is not already reported in another issue

Technical details

  • Device name: Google Pixel 3a emulator
  • Android version: 11
  • Issue first reported on App version: 1.15.1
  • Reviewed as not yet fixed in App version: 2.22.1 (except that 10 minute steps instead of 15 minute steps are allowed)

Describe the bug

It is not possible to set the duration of a visit to a place in the Contact Journal to 24 hours. The maximum possible is 23 hours and ~~45~~ 50 minutes. If somebody wants to record that they were at a place for a complete day that is not possible through the available numerical choice of hours and minutes.

Steps to reproduce the issue

  1. Open CWA, tap "Journal"
  2. Tap entry for today's date
  3. Tap or slide to "PLACES"
  4. Tap "+ PLACE"
  5. Enter text such as "Test" in Description field, tap "SAVE"
  6. Tap Duration field "00:00"
  7. Try to set the Duration to 24 hours and 00 minutes
  8. Note that the longest duration possible is 23 hours and ~~45~~ minutes. 24 is not available as a selection for hours. It is also not possible to select 23 hours and 59 minutes, because the minutes are constrained to ~~00, 15, 30 and 45~~ 00 to 50 in 10 minute increments.

max duration 23_45

Expected behavior

It should be possible to select a duration of 24 hours and 00 minutes.

Possible Fix

  • Add 24 to the list of hours available
  • Ensure that if 24 hours is selected, then only 00 minutes is considered valid input
  • Alternatively add the option to select "all day" instead of a numerical time selection

Internal Tracking ID: EXPOSUREAPP-6160

MikeMcC399 avatar Mar 26 '21 15:03 MikeMcC399

This bug is also reproducible under iOS. The maximum place duration is 23:59.

Ein-Tim avatar Mar 27 '21 23:03 Ein-Tim

@Ein-Tim

This bug is also reproducible under iOS. The maximum place duration is 23:59.

I considered 23:59 in iOS to be close enough to 24:00 for it not to matter very much, which is why I posted this issue in the cwa-app-android/issues space.

The motivation behind the bug report was considering a person attending a multi-day event, where it would be strange to have several adjacent days each recording 23:45 hours : minutes, so missing 15 minutes each day.

Another issue though is why iOS and Android are different. In Android there is no free-hand duration input, only the flippable duration picker constrained to 15 minute intervals. I guess there may have been some coding challenges which were operating system dependent. For purposes of describing how to use the Contact Journal it would be good for both operating systems to work the same way, then the instructions and screenshots would be the same, or at least very close to each other.

MikeMcC399 avatar Mar 28 '21 08:03 MikeMcC399

@MikeMcC399

Okay, understandable, so let's leave this issue here 👍

The iOS app uses the picker provided by Apple in the OS. This picker got redesigned with iOS 14 to also/mainly accept keyboard input. See this comment for more information.

Ein-Tim avatar Mar 28 '21 08:03 Ein-Tim

@MikeMcC399 @Ein-Tim Thanks. Internal Tracking ID: EXPOSUREAPP-6160

dsarkar avatar Apr 02 '21 11:04 dsarkar

@dsarkar Suggestion: Model the duration of being in a place for 24 hours using the same method that calendars, such as https://calendar.google.com/, use.

Calendars often offer the ability to select "All day" instead of specifying a numerical time.

Add a box "All day" to the journal places entry as an alternative to selecting the exact hours and minutes (currently restricted to 15 minute intervals with a maximum duration of 23h 45m).

MikeMcC399 avatar Jul 23 '21 15:07 MikeMcC399

Update: Meanwhile the highest duration which can be set under iOS is 23:50.

Ein-Tim avatar Feb 23 '22 12:02 Ein-Tim

@dsarkar Please move this issue to cwa-documentation since, according to new information from @Ein-Tim, iOS is affected as well as Android.

@Ein-Tim Thanks for the update.

MikeMcC399 avatar Feb 23 '22 13:02 MikeMcC399