maiow / dog-shelter Goto Github PK

чтобы после повторного входа юзера в приложение лайки были видны и можно было бы их ставить/убирать без прохода через экран входа/регистрации

у вас navigation bar уезжает вверх при открытии клавиатуры? Установлен adjustResize? Мне кажется, так не должно быть. – Добавлен в манифесте из-за того, что клава прикрывала кнопки на авторизации, по мне – не критично, что навбар уезжает, можно позже поправить как-то только на авторизации.

методом тыка обнаружил что из pets_card не работает боттом навигация – у меня все работает

И еще не хватает информации о приюте - название, адрес

Страница "Помочь" - добавить получателя в реквизиты Сбербанка – не уверена, что стоит, нужно уточнить у владельца счета (скорее всего, это Ира)

автоперевод текста на английский

При открытии приложения поле поиска сразу в фокусе и открыта клавиатура - это ожидаемое поведение? – какая-то фигня, у меня нет такого.

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

после запуска чата: Страница "Помочь" - рекомендуется добавить контакты или кнопку "Связаться" для быстрого перехода в мессенджер.

разделить слои так, чтобы легко перейти с файербейса на свой бэк, когда он будет,

Экран найденных: Найдено 211 в скролл нужно добавить, карточки собак как в Хоум сделать, а не сжатыми

поправить разметку всех фрагментов, привести ее к какому-то стандарту,

вместо glide плейсхолдеров подумайте на счёт шиммеров + плейсхолдер не отцентрирован – не получается его быстро отцентрировать, надо копаться, по мне – не критично

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

продолжить разделение дто-моделей: разделить для news, добавить ui модели,

переключение на Dispatcher.IO в репозитории

лупа в строке поиска кликабельной напрашивается – m3 guidelines: Icons and icon buttons, Leading icon: The left side of the bar should contain either a navigational action (such as a menu or up-arrow) or a non-functional search icon. – стоит забить на это, т.к. не по гайдлайнам

Сейчас смотрел вводную конфу у кипаро

Говорят, что в большинстве случаев фб - плохое решение

Также рассказали, что во многих компаниях, если процент кода, покрытого юнит тестами меньше какого-то порога, то даже на программном уровне не даёт сделать пулреквест

темная тема

Доработать онбординг лейаут для старых мелких экранов (см.Nexus Galaxy 4,65"). Попробовать уменьшать размер картинки в зависимости от высоты экрана.

На Хоум экране лучше продублировать кнопку "Помочь" повыше, после раздела "Новые питомцы".

экстеншен пишем для зависимостей, в котором перечисляем все, что связано с, например, хилтом, и потом просто одной строчкой вызываем -> лучше перевести на Version catalogue

карусель фулл-скрин просмотр фото собаки или возможность открытия фотографии на полный экран при нажатии

добавить авторизацию по гугл акку и/или ВК - еще какой-то второй вариант проще для юзера, чем email

Описание прилки в сторах, последний скриншот - навык денежных переводов насмешил. Про "любой навык может помочь" тут уже писали, нехорошо звучит. Любая помощь ценна или мы рады любой помощи или любой может помочь... И то что номер чьей-то карты сбера засвечен прямо на скрине в маркете - сомнительное решение. Там же есть некоторые сомнения по формулировкам - "выбирать друга или питомца по параметрам" - ну это же не телевизор, нужно выразиться более человечно.

Попросить дизайнера прислать чистую

Используйте пнгшки на картинки, если найдете. Выбиваются белые прямоугольники

фильтры сбрасываются при переходе на другую вкладку, когда делаешь поиск, и потом возвращаешься назад, то параметры поиска сбрасываются - не удобно - 3 таких комментария.

чтобы дебажные сборки не попадали в отчет

поиск - по кличке. В одной букве ошибся и всё, собака не найдена – поиск по буквам после переезда на свой сервер

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

1. Get design
2. If Ok, implement (not necessary, nice-to-have).

юнит тесты

хочется видеть графу пристроенных пёселей

Проверка наличия сети/коннекта к серверу для показа no_network. Прогресс бар и no_network из common xml добавить во все лейауты и фрагменты (в home, pets_card, favorites, profile, dogs_found уже добавлены). Нужно будет при переезде с firebase на свой сервер.

Выделение цветом той страницы, где юзер находится (вместо заполнения одним цветом по очереди)

Фича чата на stream chat sdk

поправить навигацию избранное-детали, избранное-ауф, профиль-ауф (парселайз)

Переписать базовую вью модель: при смене конфигурации или после показа no network connection и нажатия на кнопку обновить запускаются дополнительные корутины, в которых запрашиваются данные из бд, они без job все и уничтожатся только при уходе на другую вкладку боттом навигации или сворачивания прилки. Нужно будет при переезде с firebase на свой сервер.

решить: убрать или оставить алерт с предложением авторизоваться на профиле? – на алерте тот же текст, что и на экране

Мы же звоним и все такое

Авторизацию дизайн подогнать под фигму, сейчас сильно выбивается из общего. И поправить по комментам от QA: Каков корректный текст ошибки при попытке авторизоваться несуществующим пользователем? Каков корректный текст ошибки при попытке зарегистрироваться уже существующим пользователем? ( сейчас если несуществующий юзер вводит имейл, пароль и жмет кнопку войти, ему неверно показывается *проверьте состояние сети и блокируется кнопка войти). Коммент от юзера про это же: «Первый экран авторизации: Без текстовой подсказки не сразу поняла, что жму не ту кнопку) Хотела зарегистрироваться, а нажимала "войти". Стала проверять что не так с сетью (подсказка на экране: Проверьте сеть)»

"Возраст: 09.2021" это не по-русски "Родился/родилась: 09.2021" – не критично, можно оставить так, как есть, при переезде на свой сервер менять обратно не придется тогда

Проверить на реальном устройстве, если размыто - попросить дизайнера сделать лого с полями на 1/3.

Закинуть проверку на то, когда показывается боттом бар, во вью модель