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

Use Instant instead of ZonedDateTime for Point.time property #125

Closed
jenetics opened this issue Apr 23, 2020 · 1 comment
Closed

Use Instant instead of ZonedDateTime for Point.time property #125

jenetics opened this issue Apr 23, 2020 · 1 comment
Assignees
@jenetics
Labels
refactoring
Milestone
v3.0.0

Comments

@jenetics
Copy link
Owner

The Point.time() property currently uses a ZonedDateTime for storing the time. This should be replaced by the Instant class, which represents the type in the GPX file more accurately.

At a first step, the used accessors and construtors should be marked as deprecated.
@jenetics jenetics self-assigned this Apr 23, 2020
@jenetics jenetics added this to the v3.0.0 milestone Nov 13, 2021
jenetics added a commit that referenced this issue Jan 22, 2022
@jenetics
#125: Rename helper class to 'Instants'.
f6eaaf5
jenetics added a commit that referenced this issue Jan 22, 2022
@jenetics
#125: Replace 'ZonedDateTime' with 'Instant'.
2a35110
jenetics added a commit that referenced this issue Jan 22, 2022
@jenetics
#125: Fix compatibility tests.
f960716
jenetics added a commit that referenced this issue Jan 22, 2022
@jenetics
#125: Fix compatibility tests.
29dfd20
jenetics added a commit that referenced this issue Jan 22, 2022
@jenetics
#125: Optimize imports.
ed4073e
jenetics added a commit that referenced this issue Jan 22, 2022
@jenetics
#125: Update copyright year.
3ad97cb
jenetics added a commit that referenced this issue Jan 22, 2022
@jenetics
#125: Rename 'ZonedDateTimeFormat' to 'TimeFormat'.
c7d31c6
jenetics added a commit that referenced this issue Jan 22, 2022
@jenetics
#125: Additional indentations.
0e93f9b
jenetics added a commit that referenced this issue Jan 22, 2022
@jenetics
Merge pull request #160 from jenetics/issues/JPX-125-time_instant
e5b6e46 
#125: Using 'Instant' for timestamps
@jenetics
Copy link
Owner Author

Merged into r3.0.0 branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jenetics jenetics

Labels
refactoring
Projects
None yet
Milestone
v3.0.0
Development

No branches or pull requests
1 participant
@jenetics