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

Send analytic events for in-person eligibility verification / enrollment #2245

Closed
8 tasks
Tracked by #2261
angela-tran opened this issue Jul 23, 2024 · 6 comments · Fixed by #2402
Closed
8 tasks
Tracked by #2261

Send analytic events for in-person eligibility verification / enrollment #2245

angela-tran opened this issue Jul 23, 2024 · 6 comments · Fixed by #2402
Assignees
Labels
analytics App event tracking, instrumentation, logging: Metabase, Amplitude back-end Django views, sessions, middleware, models, migrations etc.

Comments

@angela-tran
Copy link
Member

angela-tran commented Jul 23, 2024

The implementation details still need to be fleshed out, but the acceptance criteria below seem correct based on dev workshop discussion on 2024-09-11.

Acceptance Criteria

  • As part of in-person eligibility verification / enrollment, analytic events are sent for:
    • started eligibility
    • returned eligibility
    • started payment connection started card tokenization
    • closed payment connection ended card tokenization
    • returned enrollment
    • failed access token request
  • Analytic events capture if the eligibility verification / enrollment is from an in-person interaction or not

Additional context

@angela-tran
Copy link
Member Author

Something I just thought of: we've talked about a feature for viewing past enrollment records, and one of the columns we're thinking we'd show is "Verified by" which is really only applicable to in-person enrollments. Is this relevant to this ticket at all? I think in general we want analytic data to be anonymized, but then we also talked about backfilling the enrollment records using analytic events and transit processor group enrollment data.

@angela-tran
Copy link
Member Author

@indexing @thekaveman and I discussed the above and decided we don't need to worry about capturing "Verified by" for this ticket or the September release.

@thekaveman
Copy link
Member

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

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

Devs suggest possible values of digital or in_person.

@thekaveman thekaveman added back-end Django views, sessions, middleware, models, migrations etc. analytics App event tracking, instrumentation, logging: Metabase, Amplitude labels Sep 12, 2024
@angela-tran
Copy link
Member Author

angela-tran commented Sep 13, 2024

Just realized this ticket needs #2244 and #2345 to be done first, so moving this to Blocked

@indexing
Copy link
Member

In this interest of more intuitive naming for parameters in our analytics tool, I propose changing method to enrollment_method. The values for this parameter remain as digital and in_person.

@angela-tran
Copy link
Member Author

@lalver1 #2382 has the code in a state that's ready to have analytics added to it. You could either build off that branch or wait for it to be merged into main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
analytics App event tracking, instrumentation, logging: Metabase, Amplitude back-end Django views, sessions, middleware, models, migrations etc.
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants