Facility page for read-only users improved

[ishanExtreme]

Issue Fixed

fixes #2509

Updates

• Created RoleButton component to render buttons according to role it supports both materalUI and html button.
• Remove/Disable "Add More Bed Types", "Add Doctor Types", "Add Triage" buttons
• Remove/Disable "Add Details of a Patient" button
• Remove/Disable "Update Facility" button
• Remove/Disable "Create Assets" button
• Remove/Disable add and update options in "Inventory Management"
• Remove/Disable "Resource Request" button
• Remove/Disable all edits option in "Bed Capacity", "Doctors List", "Corona Triage"

Task Pending

• Remove/Disable add and update options in "Location Management". In this case backend is allowing readOnly user to add location and edit it.

Demo

Corona.Care.-.29.May.2022.mp4

[ishanExtreme]

ReadOnly user credentioal
username: readonly
password: Welcome@123

[nihal467]

@ishanExtreme @gigincg test failed, the add patient button is green in the patient tab

[ishanExtreme]

@nihal467 I have modified only Facility tab, since there are many pages connected in each tab I will modify other tabs in other PRs

[ishanExtreme]

@nihal467 "Add Deatils of patient" button is diabled in facility tab since the ss you shared is patient tab, therefore its not been modified

[nihal467]

test approved, @ishanExtreme @gigincg what to do with the location management, are we making the backend change

[ishanExtreme]

@nihal467 yes I am looking into the care repo to fix this bug

[gigincg]

@nihal467 I have modified only Facility tab, since there are many pages connected in each tab I will modify other tabs in other PRs

That sounds good but, please make an issue listing out all the pages you need to complete (as checklist items), so that we don't accidentally skip out on some pages

Or if the existing issue serves this purpose you can let it be. Also please make the backend issue for the location issue

[khavinshankar]

@ishanExtreme is this pr good to be merged or is there anything pending that is to be done through this pr?
I see a pending task related to Location Management, so if my understanding is correct that will be handled through a different pr right?

[ishanExtreme]

@khavinshankar Yes this PR is good to merge but dont close the issue, I will edit the issue and make another PR once backend is fixed

[khavinshankar]

LGTM