Skip to content

LG-11185: Rename review events to enter password#9530

Merged
solipet merged 1 commit intomainfrom
dprice-lg-11185-rename-review-events
Nov 9, 2023
Merged

LG-11185: Rename review events to enter password#9530
solipet merged 1 commit intomainfrom
dprice-lg-11185-rename-review-events

Conversation

@solipet
Copy link
Contributor

@solipet solipet commented Nov 2, 2023

🎫 Ticket

LG-11185

🛠 Summary of changes

Old event names:

IdV: review info visited
IdV: review complete

The new event names now align with our typical scheme, based on the controller name:

idv_enter_password_visited
idv_enter_password_submitted

This PR in the devops repo carries these event names into existing dashboards/funnels.

@solipet solipet requested a review from a team November 2, 2023 22:07
@solipet
Copy link
Contributor Author

solipet commented Nov 2, 2023

Hold on merging until the corresponding devops PR is merged/deployed.

@solipet solipet force-pushed the dprice-lg-11185-rename-review-events branch from d078f84 to e3c2d08 Compare November 2, 2023 22:41
@solipet solipet force-pushed the dprice-lg-11185-rename-review-events branch from e3c2d08 to 22fdc09 Compare November 3, 2023 15:32
Old event names:

'IdV: review info visited'
'IdV: review complete'

The new event names now align with our typical scheme, based on the
controller name and the analytics_events method name:

:idv_enter_password_visited
:idv_enter_password_submitted

[skip changelog]

Co-authored-by: Sonia Connolly <sonia.connolly@gsa.gov>
@solipet solipet force-pushed the dprice-lg-11185-rename-review-events branch from 22fdc09 to b22a201 Compare November 3, 2023 15:41
Copy link
Contributor

@soniaconnolly soniaconnolly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@solipet
Copy link
Contributor Author

solipet commented Nov 9, 2023

Hold on merging until the corresponding devops PR is merged/deployed.

Screenshot 2023-11-09 at 1 59 05 PM

@solipet solipet merged commit 0919778 into main Nov 9, 2023
@solipet solipet deleted the dprice-lg-11185-rename-review-events branch November 9, 2023 18:59
@solipet solipet changed the title fLG-11185 Rename review events to enter password LG-11185: Rename review events to enter password Nov 9, 2023
@soniaconnolly
Copy link
Contributor

@R-Ornelas (Slightly belatedly) letting you know about a change to analytics.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants