Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • Sign in / Register
Minds Backend - Engine
Minds Backend - Engine
  • Project overview
  • Repository
  • Issues 270
  • Merge Requests 35
  • 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 2 weeks ago by Emiliano Balbuena@edgebal
Report abuse

Signup events for analytics

  • Overview 4
  • 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 2 weeks ago by Emiliano Balbuena
Request to merge goal/signup-cookie-id into master
The source branch is 2 commits behind the target branch
Open in Web IDE
Pipeline #115667418 passed for f880bb3a on goal/signup-cookie-id
          Requires approval from QA. Approved by
          Brian Hatchet
          Rami Albatal Ben Hayward Martin Santangelo Olivia Madrid Mark Harding
          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 2 weeks ago

            added scoped labels

          • Emiliano Balbuena @edgebal changed the description 2 weeks ago

            changed the description

          • Emiliano Balbuena @edgebal mentioned in issue #1240 2 weeks ago

            mentioned in issue #1240

          • Emiliano Balbuena @edgebal assigned to @benhayward.ben 2 weeks 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 · 18 hours ago
            Developer

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

          • Brian Hatchet :speech_balloon: @brianhatchet approved this merge request 18 hours ago

            approved this merge request

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

            @benhayward.ben You couldn't create a new account because I needed to create a matching front branch that had the captcha fixed.

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

            Give me a ping when this is ready Emi - I think you said you needed to add email confirmation?

          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