Skip to content

Require Product Intelligence reviews for frontend Snowplow events

What does this MR do?

Iterating on !53915 (merged) by requiring product intelligence reviews for frontend Snowplow tracking calls as well.

Related to #299123 (closed)

Testing

Commit: !54273 (82d9a9bb)

Danger bot message:

For the following files, a review from the Data team and Product Intelligence team is recommended
Please check the product intelligence guide and reach out to @gitlab-org/growth/product-intelligence/engineers group for a review.

  • app/assets/javascripts/boards/models/project.js
  • app/assets/javascripts/frequent_items/components/frequent_items_list_item.vue
  • app/assets/javascripts/sidebar/components/subscriptions/subscriptions.vue
  • app/models/project.rb
  • app/views/layouts/nav/_dashboard.html.haml
  • app/views/search/results/_wiki_blob.html.haml
  • app/views/shared/issuable/form/_type_selector.html.haml
  • ee/app/assets/javascripts/operations/components/dashboard/project.vue
  • ee/app/assets/javascripts/storage_counter/components/project.vue

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • [-] Label as security and @ mention @gitlab-com/gl-security/appsec
  • [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • [-] Security reports checked/validated by a reviewer from the AppSec team
Edited by Alishan Ladhani

Merge request reports