Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

TimeOfDayParameterType conversion doesn't work as expected. Unwanted UTC/LocalTime conversion takes place #328

Closed
3 tasks done
lxatstariongroup opened this issue Apr 9, 2024 · 0 comments · Fixed by #329
Closed
3 tasks done

TimeOfDayParameterType conversion doesn't work as expected. Unwanted UTC/LocalTime conversion takes place #328

lxatstariongroup opened this issue Apr 9, 2024 · 0 comments · Fixed by #329
Assignees
@lxatstariongroup @antoineatstariongroup
Labels
SDK

Comments

@lxatstariongroup
Copy link
Contributor

Prerequisites

  • I have written a descriptive issue title
  • I have verified that I am running the latest version of the COMET-SDK
  • I have searched open and closed issues to ensure it has not already been reported

Description

TimeOfDayParameterType conversion doesn't work as expected. Unwanted UTC/LocalTime conversion takes place in some cases.
@lxatstariongroup lxatstariongroup mentioned this issue Apr 9, 2024
Merged
4 tasks
lxatstariongroup added a commit that referenced this issue Apr 11, 2024
@lxatstariongroup
TimeOfDayParameterType conversion doesn't work as expected; fixes #328
6668b10 
TimeOfDayParameterType conversion doesn't work as expected
lxatstariongroup added a commit that referenced this issue Apr 15, 2024
@lxatstariongroup
Time of day parameter type converts to a wrong value; fixes #328
53f685e 
* Always store TimeOfDay Parameter as format 'HH:mm:ss'
* Fix unit tests
* Fix some more tests
* Fix return original value when value is correct
* Minor fix and unit tests
* Build fixed?
@samatstariongroup samatstariongroup moved this to Done in COMET-CE Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lxatstariongroup lxatstariongroup

@antoineatstariongroup antoineatstariongroup

Labels
SDK
Projects
COMET-CE
Archived in project
Milestone
No milestone
2 participants
@lxatstariongroup @antoineatstariongroup