-
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
Tailwinds / Migrates all usages of RoleButton
to ButtonV2
with authorizedFor
#5649
Conversation
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
LGTM |
@rithviknishad We truly appreciate your efforts. Thank you for taking the time to contribute; this is a very valuable contribution to us 🥇. We always welcome your contribution 🙂, so feel free to contribute to anything anytime, and never lose that spirit of innovation 🙌. |
WHAT
🤖 Generated by Copilot at 8e7b5cb
Refactored several components in the
Facility
module to use new common componentsButtonV2
andCareIcon
for improved UI and code quality. Deleted unused componentRoleButton
.Proposed Changes
RoleButton
withButtonV2
in Doctor Capacity Count CardRoleButton
withButtonV2
in Inventoy Management: Min. Quantity PageRoleButton
withButtonV2
in Location ManagementScreenshots
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist
HOW
🤖 Generated by Copilot at 8e7b5cb
BedManagement
,DoctorsCountCard
,LocationManagement
, andMinQuantityList
components to use the newButtonV2
component instead of the oldRoleButton
component (link, link, link, link, link, link, link, link)CareIcon
component to display icons instead of thei
element with font-awesome classes inBedManagement
andLocationManagement
components (link, link, link, link)React
fromreact
inLocationManagement
andMinQuantityList
components, as it is no longer needed with the new JSX transform (link, link)navigate
fromraviger
inBedManagement
,LocationManagement
, andMinQuantityList
components, as it is no longer used in the files (link, link, link)NonReadOnlyUsers
from../../Utils/AuthorizeFor
inBedManagement
,DoctorsCountCard
,LocationManagement
, andMinQuantityList
components, as it is used in the newButtonV2
component (link, link, link, link)ButtonV2
from../Common/components/ButtonV2
inBedManagement
,DoctorsCountCard
,LocationManagement
, andMinQuantityList
components, as it is used to replace the oldRoleButton
component (link, link, link, link)CareIcon
from../../CAREUI/icons/CareIcon
inBedManagement
andLocationManagement
components, as it is used to display icons (link, link)RoleButton
from../Common/RoleButton
inDoctorsCountCard
component, as it is no longer used in the file (link)src/Components/Common/RoleButton.tsx
, as it is no longer used in the project (link)