Skip to content

Next

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

Open
Opened 2 weeks ago by Marcelo Rivera@eiennohi
Report abuse

Implement new delete for channel delete to record delete event in elasticsearch

closes https://gitlab.com/minds/analytics/issues/32 closes #1093

Edited 1 day ago by Marcelo Rivera
Request to merge feat/new-delete-event-32 into master
The source branch is 5 commits behind the target branch
Open in Web IDE
Pipeline #93907404 failed for 71b075b3 on feat/new-delete-event-32
          Requires 3 more approvals from Devs, Deployers, and QA.
          Ben Hayward
          Ben Hayward
          Emiliano Balbuena
          Emiliano Balbuena
          Rami Albatal
          Rami Albatal
          Guy Thouret
          Guy Thouret
          Mark Harding
          Mark Harding
          Ready to be merged automatically. Ask someone with write access to this repository to merge this request

          Closes #1093

          Deletes source branch

          • Discussion 3
          • Commits 3
          • Pipelines 4
          • Changes 3
          1/1 thread resolved
          • Marcelo Rivera @eiennohi changed milestone to %Misc. Q4 2019 2 weeks ago

            changed milestone to %Misc. Q4 2019

          • Marcelo Rivera @eiennohi added Squad::Yellow scoped label 2 weeks ago

            added scoped label

          • Marcelo Rivera @eiennohi added 1 commit 2 weeks ago

            added 1 commit

            • 53e1f9cb - (fix): remove 'phpspec' as pagingStateToken in tests

            Compare with previous version

          • Marcelo Rivera @eiennohi added 1 commit 2 weeks ago

            added 1 commit

            • 80522e2f - (feat): add new MetricsDelegate that pushes the new delete analytics event

            Compare with previous version

          • Mark Harding
            Mark Harding @markeharding started a thread on an old version of the diff 3 days ago
            Resolved by Marcelo Rivera 1 day ago
          • Mark Harding @markeharding added MR::Requires Changes scoped label 3 days ago

            added scoped label

          • Marcelo Rivera @eiennohi added 32 commits 1 day ago

            added 32 commits

            • 80522e2f...af77626a - 30 commits from branch master
            • 9049aaef - Merge remote-tracking branch 'upstream/master' into feat/new-delete-event-32
            • 71b075b3 - (fix): metricsDelegate->onDelete() should be called outside of foreach

            Compare with previous version

          • Marcelo Rivera @eiennohi mentioned in issue #1093 1 day ago

            mentioned in issue #1093

          • Marcelo Rivera @eiennohi changed the description 1 day ago

            changed the description

          • Marcelo Rivera @eiennohi resolved all threads 1 day ago

            resolved all threads

          • Marcelo Rivera @eiennohi added MR::Awaiting Review scoped label and automatically removed MR::Requires Changes label 1 day ago

            added scoped label and automatically removed label

          • Mark Harding @markeharding added MR::Requires Changes scoped label and automatically removed MR::Awaiting Review label 51 minutes ago

            added scoped label and automatically removed label

          • Mark Harding
            Mark Harding @markeharding · 51 minutes ago
            Owner

            Linter now failing. You can get gitlab to send you emails for CI failures fyi.

          Please register or sign in to reply
          0 Assignees
          None
          Misc. Q4 2019
          Milestone
          Misc. Q4 2019
          Time tracking
          No estimate or time spent
          2
          Labels
          MR::Requires Changes Squad::Yellow
          Lock merge request
          Unlocked
          10
          10 participants
          user avatar
          Ben Hayward
          user avatar
          Emiliano Balbuena
          user avatar
          Rami Albatal
          user avatar
          Guy Thouret
          user avatar
          Mark Harding
          user avatar
          Brian Hatchet
          user avatar
          Olivia Madrid
          Reference: minds/engine!380