Skip to content
This repository has been archived by the owner on Jun 1, 2022. It is now read-only.

make combined location/report "add" form #680

Open
rhkeeler opened this issue Jun 21, 2021 · 7 comments
Open

make combined location/report "add" form #680

rhkeeler opened this issue Jun 21, 2021 · 7 comments
Labels
django-admin and tools Anything involving the /admin/ tools

Comments

@rhkeeler
Copy link

after talking to @lisabettefay we agree it would be very valuable to have a way to simultaneously add a location and file the first report for a location. We've specified the fields (and helptext) that should be on this combined form here: https://docs.google.com/document/d/178y4qVeiBQALQshbNlSA7YqaBio_jQ_F8_W-yzWNvY8/edit

Bold text = the field name should be bold
Italicized text = helptext for the field.

We're using a couple of names here that aren't the name on the location, for clarity - lmk if anything is unclear or doesn't make sense.

@rhkeeler
Copy link
Author

cc @simonw - we would like to do this, if we can! happy to talk it through with you, lmk.

@lisabettefay
Copy link

i am so excited about this. I really think this is going to "flatten the curve" for learning web banking and make it more approachable/enjoyable for people coming in totally green. THANK YOU all for working to make that happen!

@simonw simonw added the django-admin and tools Anything involving the /admin/ tools label Jun 23, 2021
@simonw
Copy link
Collaborator

simonw commented Jun 23, 2021

I'm looking at ways to do this now.

@simonw
Copy link
Collaborator

simonw commented Jun 23, 2021

I'm going to push a preview of this to staging so we can decide if it's the right direction.

@simonw
Copy link
Collaborator

simonw commented Jun 23, 2021

(The code I've written isn't ready for production yet - it needs tests, plus I need to think more carefully about edge-cases - but it's worth us looking at together in staging)

simonw added a commit that referenced this issue Jun 23, 2021
simonw added a commit that referenced this issue Jun 23, 2021
@simonw
Copy link
Collaborator

simonw commented Jun 23, 2021

You can preview what I've got so far on staging here: https://vial-staging.calltheshots.us/admin/core/location/add/?_preview=1

Scroll to the bottom to see it:

Add_location___VIAL_admin

As you can see, the appointment tag and availability tags are required fields. The vaccines available is optional.

@lisabettefay @rhkeeler are there any other report fields we should include here? We could let people populate a public not or private note for example?

@lisabettefay
Copy link

i think would still need to have:

appointments or walkins or both
appointment method
end date
public notes
private notes

if those fields are added, a full report could be entered from that combined view.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
django-admin and tools Anything involving the /admin/ tools
Projects
None yet
Development

No branches or pull requests

3 participants