Skip to content

Next

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
    • Help
    • Submit feedback
    • Contribute to GitLab
    • Switch to GitLab Next
  • Sign in / Register
Minds Backend - Engine
Minds Backend - Engine
  • Project
    • Project
    • Details
    • Activity
    • Releases
    • Dependency List
    • Cycle Analytics
    • Insights
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
    • Locked Files
  • Issues 147
    • Issues 147
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 46
    • Merge Requests 46
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Registry
    • Registry
  • Packages
    • Packages
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Minds
  • Minds Backend - EngineMinds Backend - Engine
  • Merge Requests
  • !212

Closed
Opened 2 weeks ago by Marcelo Rivera@eiennohi
  • Report abuse
Report abuse

[Sprint/GiddyGiraffe] (fix): analytics

Check out, review, and merge locally

Step 1. Fetch and check out the branch for this merge request

git fetch https://gitlab.com/eiennohi/engine.git sprint/GiddyGiraffe.fix.analytics
git checkout -b eiennohi/engine-sprint/GiddyGiraffe.fix.analytics FETCH_HEAD

Step 2. Review the changes locally

Step 3. Merge the branch and fix any conflicts that come up

git fetch origin
git checkout origin/master
git merge --no-ff eiennohi/engine-sprint/GiddyGiraffe.fix.analytics

Step 4. Push the result of the merge to GitLab

git push origin master

Note that pushing to GitLab requires write access to this repository.

Tip: You can also checkout merge requests locally by following these guidelines.

Request to merge eiennohi:sprint/GiddyGiraffe.fix.analytics into master
  • Email patches
  • Plain diff
Pipeline #69210101 passed for c9fbdee2 on eiennohi:sprint/GiddyGiraffe.fix.analytics
      Requires 2 more approvals from Devs.
      Martin Santangelo
      Martin Santangelo
      Rami Albatal
      Rami Albatal
      Emiliano Balbuena
      Emiliano Balbuena
      Ben Hayward
      Ben Hayward
      Brian Hatchet
      Brian Hatchet

      Closed by Mark Harding 57 minutes ago

      The changes were not merged into master

      Allows commits from members who can merge to the target branch

      Deletes source branch

      • Discussion 2
      • Commits 21
      • Pipelines 7
      • Changes 24
      2/2 threads resolved
      • Loading...
      • Marcelo Rivera @eiennohi added 1 commit 2 weeks ago

        added 1 commit

        • 7e71c604 - (fix): spec tests

        Compare with previous version

      • Mark Harding
        Mark Harding @markeharding started a thread on an old version of the diff 2 weeks ago
        Resolved by Marcelo Rivera 2 weeks ago
      • Brian Hatchet
        Brian Hatchet :speech_balloon: @brianhatchet started a thread on the diff 2 weeks ago
        Resolved by Marcelo Rivera 2 weeks ago
      • Marcelo Rivera @eiennohi added 2 commits 2 weeks ago

        added 2 commits

        • 892844fe - (fix): brought back active users monthly
        • 9fafccb2 - (feat): added another test

        Compare with previous version

      • Marcelo Rivera @eiennohi resolved all discussions 2 weeks ago

        resolved all discussions

      • Brian Hatchet :speech_balloon: @brianhatchet approved this merge request 2 weeks ago

        approved this merge request

      • Emiliano Balbuena @edgebal approved this merge request 2 weeks ago

        approved this merge request

      • Marcelo Rivera @eiennohi added 3 commits 1 week ago

        added 3 commits

        • ac3c73f7 - (fix): missing bracket in analytlics cli
        • f0d3e9e1 - (feat): re-add current unit
        • 51617281 - (fix): cleanup

        Compare with previous version

      • Mark Harding @markeharding added MR::Awaiting Review label 1 week ago

        added MR::Awaiting Review label

      • Marcelo Rivera @eiennohi added 6 commits 6 days ago

        added 6 commits

        • fcc54b70 - (feat): it should be 14 days vs 7
        • ba241fe5 - (fix): use correct timespans
        • f769d289 - (feat): averages for ActiveUsers
        • 3c9ca778 - (feat): averages for engagement metrics
        • 9b93e562 - (feat): averages for UserSegments
        • 1eb18b82 - (fix): correct monthly timespan values for v2/analytics

        Compare with previous version

        Toggle commit list
      • Marcelo Rivera @eiennohi added 1 commit 6 days ago

        added 1 commit

        • 1a9a4bc7 - (feat): calculate averages for everything

        Compare with previous version

      • Marcelo Rivera @eiennohi added 2 commits 5 days ago

        added 2 commits

        • a3d3f7dd - (feat): added Other category to pageviews pie chart
        • c9fbdee2 - (fix): add missing keys to UserSegments response

        Compare with previous version

      • Mark Harding @markeharding closed 57 minutes ago

        closed

      • You're only seeing other activity in the feed. To add a comment, switch to one of the following options.
      Please register or sign in to reply
      Assignee
      None
      Assign to
      None
      Milestone
      None
      Assign milestone
      None
      Time tracking
      No estimate or time spent
      1
      Labels
      MR::Awaiting Review
      Assign labels
      • View project labels
      Lock merge request
      Unlocked
      7
      7 participants
      user avatar
      Martin Santangelo
      user avatar
      Rami Albatal
      user avatar
      Emiliano Balbuena
      user avatar
      Ben Hayward
      user avatar
      Brian Hatchet
      user avatar
      Mark Harding
      user avatar
      Marcelo Rivera
      Reference: minds/engine!212