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

[TECH] - Remove name from header #543

Merged
merged 1 commit into from Sep 20, 2022

Conversation

Andyatkov
Copy link
Collaborator

@Andyatkov Andyatkov commented Sep 19, 2022

Задача Closes #0

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

Вырезается из хедера имя при создание файла

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

Ничего, просто лайк поставить

Скриншоты(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
Copy link
Collaborator

Platform: iOS
Build ID: 1597
Version: 1663593058
Internal Version: 17
Short Version: 0.0.0
Branch Name: feature/TECH-remove-name-from-header
Install URL: https://install.appcenter.ms/orgs/Admiral-Digital-Frameworks-App/apps/AdmiralUI-iOS-Dev/releases/1597

@Andyatkov Andyatkov merged commit fa821e1 into develop Sep 20, 2022
@Andyatkov Andyatkov deleted the feature/TECH-remove-name-from-header branch March 31, 2023 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants