surfstudio / ReactiveDataDisplayManager

MIT License
34 stars 13 forks source link

[SPT-1515][7.3.7] Очищать BaseEvents и сделать FoldableGenerator доступным #240

Closed Ikeret closed 1 year ago

Ikeret commented 1 year ago

Что сделано?

Зачем это сделано?

На что обратить внимание?

Как протестировать?

github-actions[bot] commented 1 year ago
Messages
:book: Great! We didn't found any violations in your changes. Congratulations 🎉

Generated by :no_entry_sign: Danger Swift against a42fc5ce6e09b47401204d8907e7f2c54bd63e4b

Ikeret commented 1 year ago

Нашел еще баг, связанный с expandable. При реюзе каждый раз новый слушатель добавляется. Недавно фиксили там краши, скорее всего из-за этого

image
codecov-commenter commented 1 year ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/7.3.7@6972981). Click here to learn what that means. Patch has no changes to coverable lines.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## release/7.3.7 #240 +/- ## ================================================ Coverage ? 61.40% ================================================ Files ? 135 Lines ? 5052 Branches ? 2282 ================================================ Hits ? 3102 Misses ? 1837 Partials ? 113 ``` | Flag | Coverage Δ | | |---|---|---| | uitests | `43.85% <0.00%> (?)` | | | unittests | `38.16% <0.00%> (?)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=surfstudio#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Ikeret commented 1 year ago

Нашел еще баг, связанный с expandable. При реюзе каждый раз новый слушатель добавляется. Недавно фиксили там краши, скорее всего из-за этого

Здесь не получится как-то исправить без управления слушателями. Есть идея добавить такой ивент:

image

При реюзе с одним и тем же id добавлять кложур связанный с ячейкой, который будет заменять старый

Но т.к. новый ивент в протоколе - повлечет изменения, лучше всего расширить функционал базовых ивентов.