[SPT-1516][7.3.7] Правка ambiguity в параметрах

Ikeret commented 1 year ago

Что сделано?

Добавлен AnimationOption для явного задания анимации
closes #236

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

Убрать ambiguity c параметром Optional.none и RowAnimation.none

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

Остальные методы не трогал, т.к. они internal и не доступны для использования

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 31864da249847a6a0156a45b4f5de727c767e89f

Ikeret commented 1 year ago

Есть в ManualTableManager много функций с передачей анимаций Хоть они там не optional, но стоит мигрировать, чтобы везде было однородно. При релизе в changelog добавить перечисление deprecated методов.

Добавлены новые методы во все менеджеры. Аниматоры не трогал, так как там публичные проперти, которые нужно менять, а не делать deprecated.

Для ситуации когда нужны 2 параметра анимации был добавлен TableRowAnimationGroup

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 #242 +/- ## ================================================ Coverage ? 61.09% ================================================ Files ? 133 Lines ? 5020 Branches ? 2268 ================================================ Hits ? 3067 Misses ? 1840 Partials ? 113 ``` | Flag | Coverage Δ | | |---|---|---| | uitests | `43.52% <0.00%> (?)` | | | unittests | `38.28% <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.

surfstudio / ReactiveDataDisplayManager