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.

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

Implement resolve_components for ZonedDateTimeFormatter #2437

Open
jedel1043 opened this issue Aug 23, 2022 · 4 comments
Open

Implement resolve_components for ZonedDateTimeFormatter #2437

jedel1043 opened this issue Aug 23, 2022 · 4 comments
Assignees
Labels
C-datetime Component: datetime, calendars, time zones S-medium Size: Less than a week (larger bug fix or enhancement) T-core Type: Required functionality

Comments

@jedel1043
Copy link
Contributor

The purpose of #1361 was to easily obtain the resolved options of a DateTimeFormatter for ECMA-402 implementations. Unfortunately, the DateTimeFormat service requires a ZonedDateTimeFormatter instead, and said component doesn't have an equivalent method to obtain its resolved options.

@sffc
Copy link
Member

sffc commented Aug 25, 2022

@jedel1043 The support for datetime components is experimental; there is a fairly clear path forward to bring them to a point where we can launch them as stable. Would you be interested in contributing to ICU4X to bring datetime components to the finish line? Happy to jump on a call to discuss.

@jedel1043
Copy link
Contributor Author

@sffc I'd love to! Would need some guidance though, since I'm still getting accustomed to the repo 😅

@sffc
Copy link
Member

sffc commented Aug 26, 2022

@jedel1043 Cool! Can you message me, sffc at google dot com, to set up some time?

@sffc sffc added this to the ICU4X 1.x Untriaged milestone Sep 8, 2022
@sffc
Copy link
Member

sffc commented Sep 8, 2022

@jedel1043 I'm assigning this to you since it is related to the components bag work.

@sffc sffc added T-core Type: Required functionality C-datetime Component: datetime, calendars, time zones S-medium Size: Less than a week (larger bug fix or enhancement) labels Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-datetime Component: datetime, calendars, time zones S-medium Size: Less than a week (larger bug fix or enhancement) T-core Type: Required functionality
Projects
None yet
Development

No branches or pull requests

2 participants