Integrate with the Amplitude analytics #365
Conversation
|
|
||
| struct ConfigureAnalyticsCommand: Command { | ||
| func execute() { | ||
| Amplitude.instance().initializeApiKey(Tokens.shared.amplitudeToken) |
There was a problem hiding this comment.
Наверное лучше убрать в ConfigureThirdPartiesCommand.swift
There was a problem hiding this comment.
Если честно, не очень понимаю, для чего нужен еще один слой абстракции в виде Command
There was a problem hiding this comment.
Это имело бы смысл, если бы нам надо было как-то персистентно сохранять команду, в противном случае все делается без этого или внутри сервиса
There was a problem hiding this comment.
@Ostrenkiy знаешь, как это бывает: начитаешься всяких статей и тянешь потом, тянешь за собой )
There was a problem hiding this comment.
Ну короче как выпилишь можно вмердживать, все события приходят
There was a problem hiding this comment.
@Ostrenkiy выпилил, теперь всё по православному 😄
|
Я, видимо, не застал ничего уже. Поэтому ок всё |
Задача: #APPS-2035
Описание:
Добавили отправку событий в Амплитуду.
Событие об отправке решения отправляется в QuizPresenter