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

reactTest #1057

Closed
wants to merge 1 commit into from
Closed

reactTest #1057

wants to merge 1 commit into from

Conversation

JoshuaVSherman
Copy link
Contributor

@JoshuaVSherman JoshuaVSherman commented Apr 5, 2018

I got it to render, but there are console warnings...
Unhandled rejection Invariant Violation: Target container is not a DOM element.
at invariant (webpack-internal:///85:42:15)
at legacyRenderSubtreeIntoContainer (webpack-internal:///214:16425:34)
at Object.render (webpack-internal:///214:16513:12)
at ReactTest.attached (webpack-internal:///react-test:40:55)
at Controller.attached (webpack-internal:///7:3539:22)
at View.attached (webpack-internal:///7:1584:23)
at ViewSlot.attached (webpack-internal:///7:1947:13)
at View.attached (webpack-internal:///7:1594:19)
at ViewSlot.attached (webpack-internal:///7:1947:13)
at eval (webpack-internal:///aurelia-framework:327:28)
From previous event:
at Aurelia.setRoot (webpack-internal:///aurelia-framework:325:40)
at eval (webpack-internal:///main:339:30)
From previous event:
at Object._callee$ (webpack-internal:///main:338:29)
at tryCatch (https://cdnjs.cloudflare.com/ajax/libs/babel-polyfill/6.26.0/polyfill.min.js:4:1423)

@JoshuaVSherman
Copy link
Contributor Author

@DannyMcwaves Also, I have linter errors, so we need to change something with linter configurations for React or something, thanks

@DannyMcwaves
Copy link
Contributor

@JoshuaVSherman Yh. I have actually seen the errors. I just wanted to make sure the functionality works first before making optimizations and fixing errors. I'm currently looking at this.

@JoshuaVSherman
Copy link
Contributor Author

@DannyMcwaves I think I just need to remove the stuff in react-test.js (I forgot to remove that)

@JoshuaVSherman
Copy link
Contributor Author

@DannyMcwaves I'll just head into work then and let you finish it up, thanks

@DannyMcwaves
Copy link
Contributor

@JoshuaVSherman great. I'll take care of it.

@JoshuaVSherman JoshuaVSherman deleted the Maria branch April 5, 2018 22:10
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 this pull request may close these issues.

2 participants