Skip to content
This repository was archived by the owner on Jan 16, 2024. It is now read-only.

Doctors assigned to other clients are listed when logged as Client #102

Merged
merged 9 commits into from
Aug 31, 2018

Conversation

xispa
Copy link
Member

@xispa xispa commented Aug 30, 2018

Description of the issue/feature this PR addresses

If the current user is a Client contact, only the doctors assigned to the same client and those that do not have any client assigned are displayed. If user is from the lab, no filtering is applied.

If the current context of the listing is Client, only doctors assigned to same client are displayed.

Dependencies:

Current behavior before PR

All doctors are displayed in Doctors listing when logged in as client contact

Desired behavior after PR is merged

Doctors not assigned to the same client as the user logged in are not displayed

--
I confirm I have tested this PR thoroughly and coded it according to PEP8
and Plone's Python styleguide standards.

xispa added 4 commits August 30, 2018 15:06
If the current user is a Client contact, only the doctors assigned
to the same client and those that do not have any client assigned are
displayed. If user is from the lab, no filtering is applied.

If the current context of the listing is Client, only doctors assigned
to same client are displayed.
Dependencies: senaite/senaite.core#1014

In AR Add form, do not display doctors from other client than the
one selected, but include those that do not have any client assigned
@xispa xispa changed the title octors assigned to other clients are listed when logged as Client Doctors assigned to other clients are listed when logged as Client Aug 30, 2018
xispa and others added 2 commits August 31, 2018 00:38
Dependencies: senaite/senaite.core#1014

In AR Add form, do not display doctors from other client than the
one selected, but include those that do not have any client assigned
@xispa xispa merged commit a92ce6b into master Aug 31, 2018
@xispa xispa deleted the client-doctor branch August 31, 2018 08:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

1 participant