-
Notifications
You must be signed in to change notification settings - Fork 434
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
fixes "Set average weekly working hour" button in user tab is not responsive Fixes #7309 #7480
fixes "Set average weekly working hour" button in user tab is not responsive Fixes #7309 #7480
Conversation
@kabirrajsingh is attempting to deploy a commit to the Open Healthcare Network Team on Vercel. A member of the Team first needs to authorize it. |
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
👋 Hi, @kabirrajsingh, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
src/Components/Users/ManageUsers.tsx
Outdated
<ButtonV2 | ||
id="avg-workinghour" | ||
className="flex w-full items-center md:w-full" | ||
className="flex w-full" |
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.
className="flex w-full" | |
className="w-full" |
src/Components/Users/ManageUsers.tsx
Outdated
@@ -410,18 +410,21 @@ export default function ManageUsers() { | |||
{["DistrictAdmin", "StateAdmin"].includes( | |||
authUser.user_type | |||
) && ( | |||
<div className="flex-col md:flex-row"> | |||
<div className=""> |
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.
<div className=""> | |
<div> |
❌ Deploy Preview for care-net failed.
|
Deploy is failing |
@kabirrajsingh fix the deploy issue |
@nihal467 it should work now. Apologies for the delay, had exams. |
it is not fixed yet |
LGTM |
@kabirrajsingh Your efforts have helped advance digital healthcare and TeleICU systems. 🚀 Thank you for taking the time out to make CARE better. We hope you continue to innovate and contribute; your impact is immense! 🙌 |
Fixes #7309
Removed useless classes in parent component and added necessary classes in child component
Screenshots attached