We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
The code proposed here UPGRADE.md for using custom notifications doesn't seem to be working.
I've added a custom notification for the demo app with the upgrade guide code here: https://codesandbox.io/s/react-admin-demo-300-gofzw
The text was updated successfully, but these errors were encountered:
Now i've added styles to avoid the errors, but the notification won't show either way.
Sorry, something went wrong.
I think the UPGRADE snippet misses the end part, which shows that you must connect the component to the Redux store:
import { connect } from 'react-redux'; import { complete, undo, getNotification, hideNotification, Button, undoableEventEmitter } from 'react-admin'; import compose from 'recompose/compose'; ... const mapStateToProps = state => ({ notification: getNotification(state), }); export default compose( withStyles(styles), connect( mapStateToProps, { complete, hideNotification, undo, } ) )(Notification);
Can you confirm that it solves the problem?
Ok, now is working. I've updated the PR.
Fixed by #3893
No branches or pull requests
The code proposed here UPGRADE.md for using custom notifications doesn't seem to be working.
I've added a custom notification for the demo app with the upgrade guide code here:
https://codesandbox.io/s/react-admin-demo-300-gofzw
The text was updated successfully, but these errors were encountered: