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

Implement the Family Planning profile #8

Open
12 tasks done
msschroeder opened this issue Nov 20, 2019 · 5 comments
Open
12 tasks done

Implement the Family Planning profile #8

msschroeder opened this issue Nov 20, 2019 · 5 comments
Assignees
Labels
bug Something isn't working family planning

Comments

@msschroeder
Copy link
Collaborator

msschroeder commented Nov 20, 2019

image

@AnnieMungai
Copy link

Ver: 1.0.2 (19th Dec 2019)
Others look good.

  • This profile should appear both from the FP register and from the All Families register. - The profile appearing on the FP register but is not appearing in all Families register.

@AnnieMungai
Copy link

If a client is on the FP and PNC register, on their All families profile, what takes precedence?

To discuss with @msschroeder and @cafootitt

@rowo
Copy link

rowo commented Dec 31, 2019

@AnnieMungai what do you mean by "precedence"? Does this mean which due buttons and rows show up first? I think those items should build as a list in their respective spots. So if a PNC visit becomes due then a FP visit, they should appear in that order. For registers that include new rows in the profile, it's chronological where additional registers get added to the next row.

@msschroeder
Copy link
Collaborator Author

@rowo @AnnieMungai @cafootitt I think Anne is saying what client profile should be used - The FP profile or PNC profile. @rowo I think it would might be helpful to have a mock-up for what this might look like, but what I would expect this to look like:

  • Top 3 dots drop down should have been accounted for in issue Update the drop down for client profile page to include family planning registration #5
  • For the text under the avatar, it would be the PNC profile, otherwise the All Families Profile text
  • Record family planning follow-up visit button row would function similar to how it would in Malaria (Implement malaria profile page with more than one visit required opensrp-client-chw-malaria#22). Ideally, I would have the button on top be the one for the oldest required visit. If not possible, I would put the PNC visit button followed by the FP visit button
  • Medical history row should use the medical history for PNC as there is no medical history for FP.
  • Family planning method row. Ideally, this would just be added to the profile if they were a FP client and not have a "new" profile for all flavors of the client profile. Not sure how we currently are doing this. @manutarus How are we doing the different profiles? Are they each defined different or using a more configurable method?
  • Upcoming services row should include both the PNC and the FP upcoming service schedules

What are your thoughts?

@rowo
Copy link

rowo commented Jan 6, 2020

I think what @msschroeder has written up is generally correct. If upcoming services needs to be split up I think that is okay too. I can provide a mockup soon.

This might take a longer discussion with an engineer like @manutarus because the multiple register thing has been done in bits and pieces. If this is indeed being built as flavors, we should talk about what the long term goal is and how it would work in a way where all profiles are built one way and things are shown or hidden based on condition.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working family planning
Projects
None yet
Development

No branches or pull requests

4 participants