6168 customised reports satisfaction rate per program #6428
Conversation
github-actions
bot
added
enhancement
| @@ -0,0 +1,52 @@ | |||
| import React from 'react'; | |||
There was a problem hiding this comment.
This whole file change was just a rename. This one only gets the Patient programs through documents
| setPeriod(period); | ||
| handleChange(path, period.id); |
There was a problem hiding this comment.
Will have to refactor this in other report issue to also return the start/end date to correctly filter out stocktake/prescriptions by start/end time
| import { PeriodFragment } from '@openmsupply-client/requisitions'; | ||
|
|
||
| export const usePeriodList = (programId?: string, enabled?: boolean) => { | ||
| const { api, storeId } = useProgramsGraphQL(); |
There was a problem hiding this comment.
Reusing the programs graphQL since they're kinda grouped together...
| @@ -22,7 +26,7 @@ pub fn programs( | |||
| let user = validate_auth( | |||
| ctx, | |||
| &ResourceAccessRequest { | |||
| resource: Resource::QueryProgram, | |||
| resource: Resource::QueryMasterList, | |||
There was a problem hiding this comment.
This is used for both Patient & Requisition programs... and doesn't make sense to use QueryProgram permission when that is a Patient Program specific permission... though it has a lot of other checks... maybe should have just created a dup with query masterlist permission? 👀
Bundle size differenceComparing this PR to
|
Fixes #6168
👩🏻💻 What does this PR do?
Review alongside https://github.com/msupply-foundation/open-msupply-reports/pull/140. Both are needed to close Satisfaction Report issue. This PR:
ProgramSearch->PatientProgramSearch🧪 Testing
📃 Documentation
1.
2.