admiral-team / admiralui-ios

Admiral UIKit iOS (https://github.com/admiral-team/admiralui-ios/wiki)

13 stars 1 forks source link

[822] - Resource Lib Remove #846

Closed Andyatkov closed 1 year ago

Andyatkov commented 1 year ago

Задача Closes #822

Что было сделано

Admiral Resource удалена из проекта

Что необходимо протестировать

Скриншоты(optional)

Review

Иван Алмазов

[ ] Все публичные свойства и методы задокументированы
[ ] Нейминг переменных и функций
[ ] Проверка на дублированость кода
[ ] Проверка на magic константы
[ ] Проверка на [weak self]
[ ] Расставлены правильно все MARK и отступы
[ ] Добавлены UI тесты на новые UI элементы
[ ] Разбитие на подфункции/на логические блоки
[ ] Перенос после "("
[ ] Проверка на грамматику
[ ] Модификаторы доступа
[ ] Проверка на неиспользуемые элементы в коде
[ ] Проверка на удаление тестированного кода

Кирилл Борисов

[ ] Все публичные свойства и методы задокументированы
[ ] Нейминг переменных и функций
[ ] Проверка на дублированость кода
[ ] Проверка на magic константы
[ ] Проверка на [weak self]
[ ] Расставлены правильно все MARK и отступы
[ ] Добавлены UI тесты на новые UI элементы
[ ] Разбитие на подфункции/на логические блоки
[ ] Перенос после "("
[ ] Проверка на грамматику
[ ] Модификаторы доступа
[ ] Проверка на неиспользуемые элементы в коде
[ ] Проверка на удаление тестированного кода

Андрей Дятков

[ ] Все публичные свойства и методы задокументированы
[ ] Нейминг переменных и функций
[ ] Проверка на дублированость кода
[ ] Проверка на magic константы
[ ] Проверка на [weak self]
[ ] Расставлены правильно все MARK и отступы
[ ] Добавлены UI тесты на новые UI элементы
[ ] Разбитие на подфункции/на логические блоки
[ ] Перенос после "("
[ ] Проверка на грамматику
[ ] Модификаторы доступа
[ ] Проверка на неиспользуемые элементы в коде
[ ] Проверка на удаление тестированного кода

Review QA

Тестирование

[ ] Проверка UI элемента
[ ] Проверка скринов после UI тестирования
[ ] Проверка в темной/светлой теме
[ ] Проверка локализации
[ ] Соотвествие дизайну

KirBorisov96 commented 1 year ago

Platform: iOS Build ID: 1930 Version: 1685019050 Internal Version: 25 Short Version: 12.0.0 Branch Name: feature/822-recources-remove Install URL: https://install.appcenter.ms/orgs/Admiral-Digital-Frameworks-App/apps/AdmiralUI-iOS-Dev/releases/1930

Andyatkov commented 1 year ago

https://github.com/admiral-team/admiralui-ios/issues/849 - завел задачу для переноса из CoreFoundation