Skip to content

Add drawerName to dataLayer pageDetails#4747

Open
lyndabanh wants to merge 1 commit intowevote:developfrom
lyndabanh:WV-4234-add-datalyer-pagedetails-drawername
Open

Add drawerName to dataLayer pageDetails#4747
lyndabanh wants to merge 1 commit intowevote:developfrom
lyndabanh:WV-4234-add-datalyer-pagedetails-drawername

Conversation

@lyndabanh
Copy link
Contributor

What github.com/wevote/WebApp/issues does this fix?

WV-4234

Changes included this pull request?

  • Add drawerName parameter to getPageDetails()
  • Update HeaderProfileDrawer and PoliticianSelfEditDrawer to pass drawerName when calling getPageDetails()
  • Implement dataLayer events in ActivityTidbitDrawer and PositionDrawer, capturing drawerName in pageDetails

Testing

  • HeaderProfileDrawer – confirmed working, drawerName appears in dataLayer
  • PoliticianSelfEditDrawer – unable to verify locally; requires a politician/candidate account
  • ActivityTidbitDrawer – unable to verify locally; onClickShowActivityTidbitDrawer is commented out in ActivityTidbitItem.jsx so the drawer cannot be opened via UI
  • PositionDrawer – unable to verify locally; could not find a UI trigger that opens this drawer in the available test environment

* Add drawerName parameter to getPageDetails()
* Update HeaderProfileDrawer and PoliticianSelfEditDrawer to pass drawerName when calling getPageDetails()
* Implement dataLayer events in ActivityTidbitDrawer and PositionDrawer, capturing drawerName in pageDetails
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.

1 participant