Skip to content
Snippets Groups Projects

Request deployment target info from users on new projects

What does this MR do and why?

This MR adds a select to the project creation form to request deployment target info from our users.

The selected option is tracked via Snowplow when the form is submitted. The event has following structure:

'select_deployment_target', {
    label: 'new_project_deployment_target',
    property: <selectedTarget>,
});

Screenshots or screen recordings

Screenshot_2022-02-03_at_19.54.49

Screenshot_2022-02-03_at_21.05.56

How to set up and validate locally

  1. Follow this guide to enable snowplow events locally
  2. Go to the Create new project -> Create blank project
  3. Validate the event being sent if the deployment target was selected and the form was submitted.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #347547 (closed)

Edited by Anna Vovchenko

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Marcia Ramos
  • Marcia Ramos
  • Marcia Ramos
  • Marcia Ramos
  • Marcia Ramos
  • Marcia Ramos
  • Marcia Ramos
  • Marcia Ramos
  • Anna Vovchenko added 1 commit

    added 1 commit

    • 7f56eac1 - Apply 9 suggestion(s) to 2 file(s)

    Compare with previous version

  • Marcia Ramos removed review request for @marcia

    removed review request for @marcia

  • Anna Vovchenko added 1 commit

    added 1 commit

    • c7d91a8e - Apply 9 suggestion(s) to 2 file(s)

    Compare with previous version

  • Anna Vovchenko requested review from @cngo and @agarciatesares

    requested review from @cngo and @agarciatesares

  • Justin Mandell approved this merge request

    approved this merge request

  • :wave: @jmandell, thanks for approving this merge request.

    This is the first time the merge request is approved. To ensure full test coverage, a new pipeline has been started.

    For more info, please refer to the following links:

  • Coung Ngo approved this merge request

    approved this merge request

  • Coung Ngo removed review request for @cngo

    removed review request for @cngo

  • Axel García approved this merge request

    approved this merge request

  • Axel García removed review request for @agarciatesares

    removed review request for @agarciatesares

  • Anna Vovchenko added 1 commit

    added 1 commit

    Compare with previous version

  • Anna Vovchenko requested review from @fneill

    requested review from @fneill

  • A deleted user added documentation label

    added documentation label

  • Fiona Neill
  • Anna Vovchenko added 1 commit

    added 1 commit

    • 4808da86 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Anna Vovchenko added 1 commit

    added 1 commit

    • 6b42135d - Implemented changes after reviews

    Compare with previous version

  • Anna Vovchenko changed the description

    changed the description

  • Anna Vovchenko requested review from @afontaine

    requested review from @afontaine

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading