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

add typescript types? #125

Closed
3 tasks done
hartzis opened this issue Feb 3, 2019 · 7 comments
Closed
3 tasks done

add typescript types? #125

hartzis opened this issue Feb 3, 2019 · 7 comments
Assignees

Comments

@hartzis
Copy link
Collaborator

hartzis commented Feb 3, 2019

@on3al
Copy link

on3al commented Feb 25, 2019

Looks like definitely typed project exists, but the types don't map up correctly 👍
https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/react-swipeable/index.d.ts

@hartzis
Copy link
Collaborator Author

hartzis commented Feb 25, 2019

@on3al awesome, thanks.

yeah, that looks like the v4 api. I updated the issue above for v5.

@hartzis hartzis added help wanted good first issue hopefully straightforward fix labels Feb 25, 2019
@hartzis hartzis self-assigned this Apr 11, 2019
@hartzis hartzis removed good first issue hopefully straightforward fix help wanted labels Apr 11, 2019
@hartzis
Copy link
Collaborator Author

hartzis commented Apr 11, 2019

Thank you @adambowles ! published #133 's types for <Swipeable/> as react-swipeable@5.2.0-alpha.0, could you check it out and verify the type additions are working?

Do you know how we deprecate the DefinitelyTyped definitions?

I'm going to attempt to create some types for useSwipeable and tests then we can get 5.2.0 out with both types.

@adambowles
Copy link
Contributor

Yep, works perfectly!

I think we should leave DefinitelyTyped up for people to get types for v4 (if they're still depending on that version for whatever reason), maybe move it into a folder to indicate this (see https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/node). I'm not very familiar with this process. Maybe we don't do anything at all.

@adambowles
Copy link
Contributor

@hartzis https://github.com/DefinitelyTyped/DefinitelyTyped#removing-a-package

@hartzis
Copy link
Collaborator Author

hartzis commented Apr 17, 2019

Going to publish these with the bug fix #138 .

All types currently published as react-swipeable@5.2.0-alpha.3.

@hartzis
Copy link
Collaborator Author

hartzis commented Apr 23, 2019

published with 5.2.0, thank you again @adambowles !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants