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

[Bug] Can't use react-plaid-link due to react-load-script issue #913

Closed
1 task
mikecousins opened this issue Feb 6, 2020 · 6 comments
Closed
1 task
Labels
bug Something isn't working external bug This issue highlights a bug in another project stale Issues that didn't get attention unreproducible This issue cannot be reproduced on master

Comments

@mikecousins
Copy link

mikecousins commented Feb 6, 2020

  • I'd be willing to implement a fix

Describe the bug

The react-plaid-link library uses react-load-script and when react-load-script is in your project it seems to kill the build.

To Reproduce

await packageJsonAndInstall({
  dependencies: {
    [`react-plaid-link`]: `1.5.0`
  }
});

Screenshots

C:\GitHub\passiv-web [master ≡ +1 ~3 -0 !]> yarn build
Creating an optimized production build...
Failed to compile.

.yarn\$$virtual\react-load-script-virtual-d93fb15f1c\0\cache\react-load-script-npm-0.0.6-df7c033ca3-2.zip\node_modules\react-load-script\lib\index.js
Module not found: A package is trying to access a peer dependency that should be provided by its direct ancestor but isn't  

command not found: react-scripts
C:\GitHub\passiv-web [master ≡ +1 ~3 -0 !]> yarn why react-load-script
├─ react-plaid-link@npm:1.5.0
│  └─ react-load-script@npm:0.0.6 (via npm:0.0.6)
│
└─ react-plaid-link@npm:1.5.0 [ed127]
   └─ react-load-script@npm:0.0.6 [cfb96] (via npm:0.0.6 [cfb96])
C:\GitHub\passiv-web [master ≡ +1 ~3 -0 !]>

Environment if relevant (please complete the following information):

  • OS: Windows 10 latest
  • Node version: 13.6.0
  • Yarn version: 2.0.0-rc28

Additional context

Add any other context about the problem here.

@mikecousins mikecousins added the bug Something isn't working label Feb 6, 2020
@yarnbot yarnbot added the unreproducible This issue cannot be reproduced on master label Feb 6, 2020
@yarnbot

This comment has been minimized.

1 similar comment
@yarnbot
Copy link
Collaborator

yarnbot commented Feb 6, 2020

We couldn't reproduce your issue (all the assertions passed on master).

@merceyz
Copy link
Member

merceyz commented Feb 6, 2020

@quinnturner
Copy link

Addressed here (waiting for merge): plaid/react-plaid-link#75

@yarnbot
Copy link
Collaborator

yarnbot commented Jun 9, 2020

Hi! 👋

This issue looks stale, and doesn't feature the reproducible label - which implies that you didn't provide a working reproduction using Sherlock. As a result, it'll be closed in a few days unless a maintainer explicitly vouches for it or you edit your first post to include a formal reproduction (you can use the playground for that).

Note that we require Sherlock reproductions for long-lived issues (rather than standalone git repositories or similar) because we're a small team. Sherlock gives us the ability to check which bugs are still affecting the master branch at any given point, and decreases the amount of code we need to run on our own machines (thus leading to faster bug resolution faster). It helps us help you! 😃

If you absolutely cannot reproduce a bug on Sherlock (for example because it's a Windows-only issue), a maintainer will have to manually add the upholded label.

@yarnbot yarnbot added the stale Issues that didn't get attention label Jun 9, 2020
@merceyz
Copy link
Member

merceyz commented Jun 10, 2020

Fixed upstream in plaid/react-plaid-link#81

@merceyz merceyz closed this as completed Jun 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working external bug This issue highlights a bug in another project stale Issues that didn't get attention unreproducible This issue cannot be reproduced on master
Projects
None yet
Development

No branches or pull requests

4 participants