Enhance `EnrollmentFlow` model with field to indicate it is a pathway that can only be done in-person

cal-itp / benefits

Transit benefits enrollment, minus the paperwork.

https://docs.calitp.org/benefits

GNU Affero General Public License v3.0

27 stars 9 forks source link

Enhance `EnrollmentFlow` model with field to indicate it is a pathway that can only be done in-person #2287

Closed angela-tran closed 1 month ago

angela-tran commented 3 months ago

Acceptance Criteria

[x] We need a way to represent that a flow is digital-only, in-person only, or supports both.

angela-tran commented 3 months ago

ignore this comment 😅

~Done shaping, need to ask for feedback~

~My thought on this is that this modeling reflects the use case we have in mind. We can refactor this field into something else if the use case changes.~

thekaveman commented 1 month ago

Let's align the field naming here with what we choose for the property name in #2245

@indexing suggests supported_methods for the name of the field/property.

Devs suggest possible values of digital or in_person. Note this should be a multi-select: one or both values can be chosen.

thekaveman commented 1 month ago

This can reuse some of the helpers introduced in #2367

E.g. benefits.core.models.EnrollmentMethods is a class with two properties for the strings digital and in_person.