-
Notifications
You must be signed in to change notification settings - Fork 0
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
Technical debts #1
Comments
Well, redux bindings are more react-ish now and waaaaay more elegant, there is a but in it, tho... const Component = {
props: connect(store, ({ myReducer, myOtherThing } ) => ({ myReducer, myOtherThing })),
mergedThings: ({ props }) => props.myReducer + props.myOtherThing,
} Maybe put this "mergedThings" logic inside my redux binding for Hybrids. I hope it works |
E2E tests are out of league of now. Cypress can't access shadowDom for now. Maybe Selenium or WebDriver could do the trick, but I wanted something to make E2E tests easier, and Cypress definitely is the main choice. |
"Route components MUST NOT be containers, they are just High Order Components, and their object is just to compose the view." |
So, stuff got a little different, some of the business logic are being changed
|
Okay, now that I'm not beeing pushed to deploy it ASAP, I can start to take care of the things i've done. I'll try to make a more react-ish approach:
And if everything is alright, I can think in something to test the components, something like Cypress
The text was updated successfully, but these errors were encountered: