Skip to content

Rename `suggestions_requested ` to `suggestion_requested`

François Rosé requested to merge francoisrose/2023-10-27-fix-snowplow-action-name into main

What does this merge request do and why?

The Snowplow event sent by the AI Gateway has a name that is inconsistent with the equivalent event sent by the IDE Extensions. See telemetry specs here.

This MR changes the action name from the plural form to the singular form.

See background and details in https://gitlab.com/gitlab-org/modelops/applied-ml/code-suggestions/ai-assist/-/issues/333+.

How to set up and validate locally

I don't know how to test this locally, and I need help with this!

Note that the existing Snowplow test already uses the correct action name, even though the actual name was different. Which suggests that there isn't any automated testing in place that checks the event name itself.

Merge request checklist

  • Tests added for new functionality. If not, please raise an issue to follow up.
  • Documentation added/updated, if needed.

Merge request reports