Skip to content
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

Reading appointments as clinician #143

Closed
1 task done
arkadiuszbachorski opened this issue Sep 24, 2024 · 0 comments · Fixed by #144
Closed
1 task done

Reading appointments as clinician #143

arkadiuszbachorski opened this issue Sep 24, 2024 · 0 comments · Fixed by #144
Assignees
Labels
bug Something isn't working

Comments

@arkadiuszbachorski
Copy link
Contributor

arkadiuszbachorski commented Sep 24, 2024

Description

I am having a trouble reading /users/{userId}/appointments as a clinician. I get:

evaluation error at L146:22 for 'list' @ L146, Property organization is undefined on object. for 'list' @ L146
The problem is somewhere around here:

allow read: if isAdmin()
|| isUser(userId)
|| isOwnerOrClinicianOfSameOrganization();

I was trying to troubleshoot, but unsuccessfully. It seems correct. It happens on appointments view, which used to work previously, so it might be some kind of regression.

Reproduction

  1. Create environment with seed data (I used this branch Seed clinician messages #142 )
  2. Request /users/{userId}/appoinments as clinician.

Expected behavior

If userId belongs to the same organization, they should be able to read data

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct and Contributing Guidelines
@arkadiuszbachorski arkadiuszbachorski added the bug Something isn't working label Sep 24, 2024
@arkadiuszbachorski arkadiuszbachorski changed the title Read of appointments Reading appointments as clinician Sep 24, 2024
@pauljohanneskraft pauljohanneskraft linked a pull request Sep 24, 2024 that will close this issue
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants