-
Notifications
You must be signed in to change notification settings - Fork 33
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
Allow paying with Edenred #1825
base: master
Are you sure you want to change the base?
Conversation
f39b534
to
977fd9d
Compare
) | ||
} | ||
|
||
function mapStateToProps(state) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we should discuss it first, but for me, using useSelector
and useDispatch
makes code easier to understand, rather than scrolling back and forth to know what each prop means. See, for example: https://github.com/coopcycle/coopcycle-app/blob/645241035b8e9bd96eb481a3b483aee1f44f2280/src/navigation/account/RegisterConfirm.js
@alexsegura please check the tests before merging if some of them have a valid reason to fail. And I'll test again when we have a production build ready |
Fixes #1775