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

Added in timezone string property to timepicker block element in types package #1503

Merged
merged 1 commit into from Jul 7, 2022

Conversation

hello-ashleyintech
Copy link
Contributor

Summary

As mentioned in Issue #1502, we need to add a new optional String property, timezone, for the Timepicker element. This needs to be added to the @slack/types package.

Requirements (place an x in each [ ])

@hello-ashleyintech hello-ashleyintech added enhancement M-T: A feature request for new functionality area:typescript issues that specifically impact using the package from typescript projects pkg:types applies to `@slack/types` labels Jun 29, 2022
@seratch seratch added this to the types@2.6.0 milestone Jun 29, 2022
Copy link
Member

@seratch seratch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me but we should hold off merging this until the timing that the property is publicly available in the API documents.

@seratch
Copy link
Member

seratch commented Jun 29, 2022

For the above reason, let me change this PR to draft mode.

@seratch seratch marked this pull request as draft June 29, 2022 21:20
@hello-ashleyintech
Copy link
Contributor Author

Sounds good to me! Thanks, Kaz! @seratch 🙌

@seratch seratch modified the milestones: types@2.6.0, types@2.7.0 Jul 4, 2022
@seratch seratch marked this pull request as ready for review July 7, 2022 04:43
@seratch
Copy link
Member

seratch commented Jul 7, 2022

The server-side engineering team told me that this feature is ready for all clients now. Also, the API document site will be updated soon. Let me merge this PR now. Thanks for your contribution! @hello-ashleyintech

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:typescript issues that specifically impact using the package from typescript projects enhancement M-T: A feature request for new functionality pkg:types applies to `@slack/types`
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants