Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • Sign in / Register
Minds Backend - Engine
Minds Backend - Engine
  • Project overview
  • Repository
  • Issues 272
  • Merge Requests 34
  • CI / CD
  • Security & Compliance
  • Packages
  • Analytics
  • Wiki
  • Snippets
  • Members
  • Collapse sidebar
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Minds
  • Minds Backend - EngineMinds Backend - Engine
  • Merge Requests
  • !450

Open
Opened 1 week ago by Emiliano Balbuena@edgebal
Report abuse

Signup events for analytics

  • Overview 2
  • Commits 2
  • Pipelines 2
  • Changes 1

Summary

Push signup events to analyze user conversions

  • Closes #1240

Steps to test

  • Port forward sandbox sites elasticsearch cluster to your local machine (optional)
  • Open your elasticsearch browser and open minds-metrics-* index
  • Sort by @timestamp descending
  • Open the review site, and create a new user. Don't do anything else after that to avoid unnnecesary events.
  • Refresh your minds-metrics-* view on ElasticSearch browser
  • Ensure there's a signup metric with a user_guid and a cookie_id value set (might be the third or fourth row from the top). Filtering by cookie_id (which has the same value a mwa cookie on the website).

Note: This one might be tricky for @xander-miller to test as it's a backend-only storage change. The main concern is to ensure new user registration works OK.

Example data: image

Estimated Regression Scope

New block of code executed at register/complete event, might affect user sign-up.

Edited 1 week ago by Emiliano Balbuena
Request to merge goal/signup-cookie-id into master
Open in Web IDE
Pipeline #115667418 passed for f880bb3a on goal/signup-cookie-id
          Requires approval from QA. Approved by
          Brian Hatchet
          Martin Santangelo Rami Albatal Olivia Madrid Mark Harding Ben Hayward
          Ready to be merged automatically. Ask someone with write access to this repository to merge this request

          Closes #1240

          Deletes source branch

          • Emiliano Balbuena @edgebal added Squad::Yellow Status::Awaiting Review scoped labels 1 week ago

            added scoped labels

          • Emiliano Balbuena @edgebal changed the description 1 week ago

            changed the description

          • Emiliano Balbuena @edgebal mentioned in issue #1240 1 week ago

            mentioned in issue #1240

          • Emiliano Balbuena @edgebal assigned to @benhayward.ben 1 week ago

            assigned to @benhayward.ben

          • Ben Hayward
            Ben Hayward @benhayward.ben · 1 week ago
            Developer

            @Xander if you need help getting set-up to test this one also, DM me.

          • Mark Harding @markeharding approved this merge request 1 week ago

            approved this merge request

          • Emiliano Balbuena @edgebal changed target branch from master to production 1 week ago

            changed target branch from master to production

          • Emiliano Balbuena @edgebal added 21 commits 5 days ago

            added 21 commits

            • dcaa9f5d...fec60070 - 4 commits from branch production
            • de6fd8ed - (wip): Frontend rendering module
            • 7e2c2364 - Merge branch 'master' of https://gitlab.com/Minds/engine into epic/SSR
            • dcaea0d4 - (chore): ci fixes
            • f2751134 - Merge branch 'master' of https://gitlab.com/Minds/engine into epic/SSR
            • 9a498785 - (chore): ci for preprod
            • 5562dca5 - (chore): export pro on api config
            • 1d9730a6 - Merge branch 'master' of https://gitlab.com/Minds/engine into epic/SSR
            • 1ccf1085 - (chore): allow master to push to preprod too
            • d3b9a8ec - Merge branch 'epic/SSR'
            • e4fdcb9e - (chore): january token gift
            • 550c417c - Merge branch 'production' of https://gitlab.com/Minds/engine
            • 85275399 - (chore): bring sandbox helm chart branch to master
            • 2d4ca7d6 - (feat): backend for captcha - front#646 (closed)
            • 7c362781 - Merge branch 'feat/646-captcha' into 'master'
            • 5c3e5b05 - Add environment support to Feature services
            • 5e9b7bb9 - Merge branch 'goal/unleash-sync-namespaces' into 'master'
            • f880bb3a - Merge remote-tracking branch 'origin/master' into goal/signup-cookie-id

            Compare with previous version

            Toggle commit list
          • Emiliano Balbuena @edgebal changed target branch from production to master 5 days ago

            changed target branch from production to master

          • Emiliano Balbuena
            Emiliano Balbuena @edgebal · 16 minutes ago
            Developer

            @benhayward.ben I just re-started review site for this MR.

          • Brian Hatchet :speech_balloon: @brianhatchet approved this merge request 8 minutes ago

            approved this merge request

          Please register or sign in to reply
          2 Assignees
          Emiliano Balbuena's avatar
          Ben Hayward's avatar
          None
          Milestone
          None
          Time tracking
          No estimate or time spent
          2
          Labels
          Squad::Yellow Status::Awaiting Review
          Lock merge request
          Unlocked
          10
          10 participants
          user avatar
          user avatar
          user avatar
          user avatar
          user avatar
          user avatar
          user avatar
          Reference: minds/engine!450