(cache)Implement new delete for channel delete to record delete event in elasticsearch (!380) · Merge Requests · Minds / Minds Backend - Engine · GitLab

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

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

Edited 14 minutes ago by Marcelo Rivera

Request to merge feat/new-delete-event-32 into master

Open in Web IDE

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 @markeharding started a thread on an old version of the diff 1 day ago

Resolved by Marcelo Rivera 14 minutes ago
Mark Harding @markeharding added MR::Requires Changes scoped label 1 day ago

added scoped label
Marcelo Rivera @eiennohi added 32 commits 15 minutes 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 15 minutes ago

mentioned in issue #1093
Marcelo Rivera @eiennohi changed the description 14 minutes ago

changed the description
Marcelo Rivera @eiennohi resolved all threads 14 minutes ago

resolved all threads
Marcelo Rivera @eiennohi added MR::Awaiting Review scoped label and automatically removed MR::Requires Changes label 14 minutes ago

added scoped label and automatically removed label

Please register or sign in to reply