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

Using EUI 13.x in a standalone project results in import errors #2271

Closed
dvas0004 opened this issue Aug 29, 2019 · 1 comment · Fixed by #2279
Closed

Using EUI 13.x in a standalone project results in import errors #2271

dvas0004 opened this issue Aug 29, 2019 · 1 comment · Fixed by #2279
Assignees
Labels

Comments

@dvas0004
Copy link

To reproduce:

  1. create-react-app <your_project>
  2. yarn add @elastic/eui @elastic/datemath moment
    (in my case this installs v13.7.0)
  3. copy/paste some boilerplate code. I used https://elastic.github.io/eui/#/layout/page
  4. yarn start

For versions 13.7.0, 13.6.0, 13.0.0 this gives import errors such as
buttonColor is not exported from button.js

Note:

  • this doesnt happen for v12.4.0 - works perfectly fine
  • i dont know if this makes a difference but I didnt test with typescript, i used plain JS
@dvas0004 dvas0004 changed the title Using EUI 13.x in a standalone project results in errors Using EUI 13.x in a standalone project results in import errors Aug 29, 2019
@thompsongl
Copy link
Contributor

thompsongl commented Aug 29, 2019

Thanks for the report, @dvas0004

I do see the type-related import error(s) when using 13.7.0 in CRA, but not in any versions before that. We made some pretty substantial TypeScript changes that were released in 13.7.0, so I'd be surprised if the problem didn't come from that version.

Until we get it patched, try using 13.6.1 and pinning to that version ("13.6.1" instead of "^13.6.1") and yarn --force to ensure 13.7.0 is not still the installed version.

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

Successfully merging a pull request may close this issue.

2 participants