Automattic / bugomattic

Bugomattic is a tool that guides bug reporters to the right actions within large, complex organizations
GNU General Public License v2.0
6 stars 0 forks source link

Add track events #108

Closed karenroldan closed 1 year ago

karenroldan commented 1 year ago

What Does This PR Add/Change?

Add track events based on pciE2j-27d-p2

  1. issue_search
  2. status_filter_select
  3. repo_filter_select
  4. sort_select
  5. issue_link_click
  6. navbar_report_issue_start
  7. banner_report_issue_start
  8. navbar_item_click

Testing Instructions

Issues

Related to #
Closes #

john-legg commented 1 year ago

This looks great, Karen! The code compiled properly and all of the track events work as expected 😄

suggestion (non-blocking): I did want to follow up on one note I made in the previous comment

This made me realize that we've been a bit inconsistent with our naming of this field because even the duplicate search result row uses Opened 4 days ago lol.

For the duplicate search result rows, would it be possible to change the word Opened to Created to better match the renamed Date Created sorting option? This isn't urgent, so it can be in a separate PR if the word change sounds reasonable to you!

karenroldan commented 1 year ago

For the duplicate search result rows, would it be possible to change the word Opened to Created to better match the renamed Date Created sorting option?

Ofc, I missed the previous text! 🤦🏻‍♀️ it should be fixed in #113. Thanks again for checking.