opensrp / opensrp-client-chw-emergency-transport

CHW Emergency Transport
0 stars 0 forks source link

Ad the GPS point of house location on geowidget - CHW #2

Open msschroeder opened 4 years ago

msschroeder commented 4 years ago

NOTE: Since the GPS is not required, a client may not have a GPS associated with themselves.

githengi commented 4 years ago

LOE 2 days for 1 FTE

rowo commented 4 years ago

@msschroeder looking at this issue first regards to icon (which is related to https://github.com/OpenSRP/opensrp-client-chw-emergency-transport/issues/4 and https://github.com/OpenSRP/opensrp-client-chw-emergency-transport/issues/3), so I'll put my questions here.

  1. Just to confirm, there is only one screen that shows the current location of the user's device, household point (single point), the responders (of which there can be up to 1 to 7, and health facility (of which there are 7 types).
  2. Will the user will be able to zoom in/out so more items could appear? How is the default zoom level set?
  3. Is the primary purpose of this screen to determine the location of the house and contact the closest emergency transport, letting them know which facility to go to? Is there any other reason for this screen?
  4. Do the different types of health facility need to be differentiated down to the specific of 7 types? This seems a bit overkill. Perhaps if they can be categorized into two buckets (like where "water points" are sometimes improved vs. not), or just treated the same until you tap, that could be clearer.

I think we should use both icons and colors to make it clear. For context, Reveal (and as a result geowidget), uses colors to represent task status (like "not done" and "done") with icons for some special points of interest like water points, but mostly geometric shapes for the residential structures.

msschroeder commented 4 years ago

@rowo To answer your questions, please see below:

  1. Just to confirm, there is only one screen that shows the current location of the user's device, household point (single point), the responders (of which there can be up to 1 to 7, and health facility (of which there are 7 types).

Yes, that is correct. On screen for all those features: user's device, household point, responders (of which there can 1 to 7), and the health facility types

  1. Will the user will be able to zoom in/out so more items could appear? How is the default zoom level set?

That is a good question. @githengi @manutarus What are your recommendations in regards to the zoom for the geowidget?

  1. Is the primary purpose of this screen to determine the location of the house and contact the closest emergency transport, letting them know which facility to go to? Is there any other reason for this screen?

Yes, that is the primary purpose of the screen.

  1. Do the different types of health facility need to be differentiated down to the specific of 7 types? This seems a bit overkill. Perhaps if they can be categorized into two buckets (like where "water points" are sometimes improved vs. not), or just treated the same until you tap, that could be clearer.

The different type of 7 facilities makes sense to me. I will check with the client and see if they want to highlight the facility or just certain services. My understanding was the different facilities.

githengi commented 4 years ago

The default zoom level can be resolved by checking the size of the catchment areas so that by default it can show the whole catchment area.

The map should also be centered on the house.

There are still edge cases

msschroeder commented 4 years ago

@githengi Are the edge cases more limitations or do we need to have worry about them technically? My understanding is they are more limitations

githengi commented 4 years ago

These would need to be considered when deciding the zoom level as the map can and should behave differently for those different conditions

rowo commented 4 years ago

For zoom, I kind of wonder whether we should default to show the user or not the user location. If the use case is primarily the mother will be at home, then we could just use that. If we think it's very unlikely the user will be with the mother, but away from the house, then we should include the user in the zoom.

@msschroeder I know we chatted on Slack, but not sure if there was a resolution to how much we need to differentiate the different facilities in https://github.com/OpenSRP/opensrp-client-chw-emergency-transport/issues/3. Seven different styles (indicating the 7 types) but all within the same family (health facility) seems a bit overkill. Were you able to chat with the client to see if we can reduce the number down to maybe two categories (for example "good" vs "not as good" place to deliver)?

msschroeder commented 4 years ago

@rowo For zoom, I think we should default to NOT include user location because the user will probably not be near the mother.

For the issue of the facilities - I think we should differentiate between the facilities CEmONC = yes and CEmONC = no as these are focused on emergency services that would be important to a delivery mother in such contexts I imagine this component of the app would be used

Mstjamush commented 4 years ago

@manutarus change the colour of the client's household geo locator from red to #fff85f (a non-blinding yellow color)