-
Notifications
You must be signed in to change notification settings - Fork 347
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
Adds Illustration when no notices available + Replaced Material UI components in NoticeBoard page #5552
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
@rithviknishad can you please check the PR |
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.
Why are changes from #5544 present here?
Ensure your changes from other branches are not present.
yeahh ill change that |
@rithviknishad can you check if this is okay ! |
The changes from other branch are still present. Feel free to create a new branch and re-open another PR. This can mess up when the other PR and this PR merges. |
Or you could make these changes in the other PR itself since that already removes some of the mui componennts, and link 2 issues to that PR if you are facing difficulty. |
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.
Use Care Icon's bell-slash
icon instead of illustration.
changes requested by @nihal467
|
sure @nihal467 ill work on it |
@nihal467 @rithviknishad do we already have bell-slash in care icons ? |
@nikhilagastya all unicons are present in care icons. you just need to prefix the classname like: |
@rithviknishad kindly check if that works ! |
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.
Awesome!
LGTM, great work @nikhilagastya |
Kudos, SonarCloud Quality Gate passed!
|
Created custom components by removing Material UI
src/Components/Notifications/NoticeBoard.tsx
: #4973