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

Error: Unable to resolve module ./Interop #38

Closed
Freddy03h opened this issue Aug 21, 2020 · 6 comments · Fixed by #39
Closed

Error: Unable to resolve module ./Interop #38

Freddy03h opened this issue Aug 21, 2020 · 6 comments · Fixed by #39

Comments

@Freddy03h
Copy link
Member

New issue since 5.0.1

error: Error: Unable to resolve module `./Interop` from `node_modules/reason-react-navigation/src/Stack.bs.js`:

I think it's relative to this commit because I didn't get this issue on my branch : 64d12bb

@MoOx
Copy link
Member

MoOx commented Sep 11, 2020

5.0.2 is out to fix this

@pedrobslisboa
Copy link

@MoOx I'm still with this issue at 5.1.3

image

@MoOx
Copy link
Member

MoOx commented Jun 15, 2021

That's unexpected because file is actually available in the npm package https://unpkg.com/[email protected]/src/Interop.js
Can you investigate @pedrobslisboa ?

@Freddy03h
Copy link
Member Author

Freddy03h commented Jun 15, 2021

It reminds me some issue I've got before I configured "in-source": true in bsconfig, after upgraded bs-platform

@jaredramirez
Copy link

We have a 100% rescript app, so we don't use "in-source": true. To get around this, we have have our scripts setup to run:

mkdir -p node_modules/rescript-react-navigation/lib/es6/src && cp node_modules/rescript-react-navigation/src/Interop.js node_modules/rescript-react-navigation/lib/es6/src/Interop.js

before every rescript build to make everything work.

Though, since Interop only exposes identity, I don't see why not just use the type conversion escape hatch.

@MoOx
Copy link
Member

MoOx commented Mar 7, 2022

@jaredramirez PR welcome if you find a way to remove this Interop file. I tried back then but didn't succeed.

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

Successfully merging a pull request may close this issue.

4 participants