Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[326, 336] - Elements & buttons fix #626

Merged
merged 1 commit into from Nov 9, 2022

Conversation

Almazovivan
Copy link
Collaborator

@Almazovivan Almazovivan commented Oct 31, 2022

Задача Closes #326

#326
#528

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

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

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

Review

Иван Алмазов

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

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

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

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

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

Review QA

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

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

@Almazovivan Almazovivan added the bug Something isn't working label Oct 31, 2022
@Almazovivan Almazovivan added this to the Release 18 milestone Oct 31, 2022
@Almazovivan Almazovivan self-assigned this Oct 31, 2022
@KirBorisov96
Copy link
Collaborator

Platform: iOS
Build ID: 1680
Version: 1667219842
Internal Version: 17
Short Version: 0.0.0
Branch Name: feature/326-336-elements-buttons-fix
Install URL: https://install.appcenter.ms/orgs/Admiral-Digital-Frameworks-App/apps/AdmiralUI-iOS-Dev/releases/1680

@KirBorisov96 KirBorisov96 modified the milestones: Release 18, Release 19 Nov 9, 2022
@Almazovivan Almazovivan merged commit e592f46 into develop Nov 9, 2022
@Andyatkov Andyatkov deleted the feature/326-336-elements-buttons-fix branch March 31, 2023 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] - Корректировка разделов Buttons
3 participants