KT
я вот вас добавил в тестовый проект, чтобы посмотреть, что будет за ивент. Потом отпишитесь пожалуйста.
Size: a a a
2017 November 19
ES
спс
ES
+ там должен ещё появиться, когда у вас доступ закончиться
ES
через 4 часа
KT
2017 November 20
ES
@terrakok
я видел, что вы влили мой Todo pr. Мне теперь нужно сделать новый на develop?
я видел, что вы влили мой Todo pr. Мне теперь нужно сделать новый на develop?
KT
ES
ок
KT
Хотя можно подождать дизайна
ES
давайте так и сделаем это более логично. Я тогда и дизайн заоодно добавлю.
2017 November 21
ES
@terrakok
добрый вечер. Посмотрите пожалуйста mr по event.
добрый вечер. Посмотрите пожалуйста mr по event.
2017 November 22
KT
смотрю
KT
1) зачем вы перенесли EventAction.kt и удалили от туда половину ключей? этот список прямиком из документации АПИ. и в апп ентити никаких json полей быть не может!
2) EventType - а по содержимому это EventTarget (и запятая лишняя)
3) FullEventInfo содержит поле Project, но не каждый евент относится к проекту
4) val targetIid: Long? - лучше просто val targetId и уж точно не нуллабл
2) EventType - а по содержимому это EventTarget (и запятая лишняя)
3) FullEventInfo содержит поле Project, но не каждый евент относится к проекту
4) val targetIid: Long? - лучше просто val targetId и уж точно не нуллабл
KT
извиняюсь, что сюда, но сейчас так удобнее
ES
1) перенес его туда, т.к он используется в FullEventInfo, удалил половину, т.к они больше не приходят. Но какие-то верну. Посмотрите есть ли у вас expired event недавний.
2) это наш локальный, в EventTarget нет Commit, Branch, Project, т.к в этом случае приходит null
3) а какой ивент не относится к проекту?
4) насчёт названия согласен, но вот тут не каждому ивенту нужен этот id (Branch, Commit)
2) это наш локальный, в EventTarget нет Commit, Branch, Project, т.к в этом случае приходит null
3) а какой ивент не относится к проекту?
4) насчёт названия согласен, но вот тут не каждому ивенту нужен этот id (Branch, Commit)
ES
KT
перенес его туда, т.к он используется в FullEventInfo - это не повод. оставьте где было и согласуйте с документацией. удалять ничего не надо
ES
но вот pushed не приходит, вместо него теперь pushed to, pushed new
KT
KT
но переносить нельзя