Need .d.ts declaration about hooks

[sarink]

Why is this not published yet??

[jrwpatterson]

is anybody dealing with this?

[zhigang1992]

For anyone that want to use it right now, you can use this one. 👇

package.json

{
   ...
    "react-navigation-hooks": "npm:@zhigang1992/react-navigation-hooks@latest",
   ...
}

[bySabi]

Great!
I hope to have some PR soon.

[slorber]

I merged the PR here and you can get the types starting 1.0.2-alpha.0

[slorber]

Yep, types are not published:
https://unpkg.com/[email protected]/dist/Hooks.d.ts

we can use types or typings with TS

[bySabi]

Created a first PR on typescript environment and tooling only: #10

[zaguiini]

Can confirm that republishing works. It looks like the published version was using another workflow instead of the most recent one, so the types were not being generated/published.

I made a simple test that proves that (note the -test suffix on the package's name):

I really just ran yarn publish.

[slorber]

@brentvatne @satya164 hi, do you have publish access on this repo?

[slorber]

Hey, As ReactNavigation will be in typescript (i think?) maybe we should just write this lib in typescript too?

Btw why not use @react-navigation/hooks package name? Is this package different from other navigation packages?

[ericvicenti]

TS support would definitely be an appreciated contribution!

The reasoning behind the react-navigation-hooks name is that it is a testing ground for now, before hooks are stable. Once react-nav starts depending on react 16.7+, we can move this to /core.

[bySabi]

@ericvicenti I want to contrib rewriting this package in typescript. I would like to know if there is any problem in upgrade it to babel 7 ?

[ericvicenti]

Feel free to upgrade us to babel 7, I think that would be great

[bySabi]

Now is done. Waiting for PR get merged.
I'm planing to add a example repo to.

[zaguiini]

Can't use it yet. Am I doing something wrong? There's no .d.ts file

[kevinwolfcr]

@zaguiini this is happening because the property on the package.json should be typings and currently is types. I've created #15 to fix that.

[zaguiini]

I don’t think so, @kevinwolfcr, because some packages I publish to npm doesn’t have this field too and the typings work. Probably the d.ts files aren't published

[slorber]

@ericvicenti could you republish with types? Or share publish right with me or someone else if you are busy? thanks

[zaguiini]

is anybody dealing with this?

Best you can do right now is to create a declaration file by yourself inside your project...

[shamilovtim]

Still need types please!

[slorber]

Hi,

Can you try this please?

yarn add [email protected]

I've made a PR to update the TS env to use babel ts plugin through Bob. It's not yet merged yet (because I have a problem setting up jest/tests) but still made a release so that people here can try the ts types.

#27