-
Notifications
You must be signed in to change notification settings - Fork 54
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
Replace showToast to use showSnackbar #42
Conversation
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.
Thanks @sherrif10
package.json
Outdated
"devDependencies": { | ||
"@ohri/openmrs-esm-ohri-commons-lib": "next", | ||
"@openmrs/esm-extensions": "next", | ||
"@openmrs/esm-framework": "^5.3.2", |
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.
Was this intentional? or did yarn do this? typically we use next
tag so even after the yarn up
it's better to leave it as next
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.
Yarn did this. Let me get it back
package.json
Outdated
"jest-cli": "^28.1.3", | ||
"jest-environment-jsdom": "^28.1.3", | ||
"lerna": "^5.6.1", | ||
"openmrs": "^5.3.2", |
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.
same applies here
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.
Thanks @sherrif10
Requirements
Summary
Screenshots
Related Issue
Other
This PR replaces showToast with showSnackbar
Before
After
cc @pirupius