Feature/76 edit user profile #87
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
artwist-polyakov
requested changes
Apr 22, 2024
There was a problem hiding this comment.
Одно замечание.
По реализации лично мне кажется странным, что мы используем специальные типы ошибок для редактирования профиля. Такие эксепшены кажутся универсальными (вроде нет соединения или анаутентификейтед), но так как нас поджимают сроки мы вряд ли сможем организовать работу сетевого слоя красиво.
app/src/main/java/app/cashadvisor/profile/data/impl/ProfileInfoRepositoryImpl.kt
Show resolved
Hide resolved
app/src/main/java/app/cashadvisor/profile/data/mapper/NetworkToProfileExceptionMapper.kt
Outdated
Show resolved
Hide resolved
avanisimov
reviewed
Apr 23, 2024
uikit/src/main/java/app/cashadvisor/customView/ProfileHeaderMain.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/app/cashadvisor/profile/data/api/ProfileInfoApiService.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/app/cashadvisor/profile/data/impl/ProfileInfoRepositoryImpl.kt
Show resolved
Hide resolved
app/src/main/java/app/cashadvisor/profile/data/impl/ProfileInfoRepositoryImpl.kt
Show resolved
Hide resolved
app/src/main/java/app/cashadvisor/profile/di/ProfileDataModule.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/app/cashadvisor/profile/di/ProfileDataModule.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/app/cashadvisor/profile/di/ProfileDataModule.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/app/cashadvisor/profile/presentation/ui/ProfileSettingsFragment.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/app/cashadvisor/profile/presentation/ui/ProfileSettingsFragment.kt
Outdated
Show resolved
Hide resolved
…ing lifetime of the application
# Conflicts: # app/src/main/res/navigation/nav_graph.xml # uikit/src/main/res/values/dimens.xml
avanisimov
reviewed
May 5, 2024
app/src/main/java/app/cashadvisor/profile/presentation/ui/ProfileSettingsFragment.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/app/cashadvisor/profile/presentation/ui/ProfileSettingsFragment.kt
Outdated
Show resolved
Hide resolved
# Conflicts: # uikit/src/main/res/values/dimens.xml # uikit/src/main/res/values/strings.xml # uikit/src/main/res/values/styles_btn.xml
avanisimov
approved these changes
May 20, 2024
artwist-polyakov
approved these changes
May 20, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Критерии приемки:
Доступные действия:
изменение имени и/или фамилии пользователя
добавление / изменение изображения (клик по рамке изображения)
возврат на предыдущий экран (кнопка назад)
кнопка "Сохранить" всегда активна.
Требования к тексту: 1-50 символов латинского или русского алфавита (^[a-zA-Zа-яА-Я]{1,50}$)
После нажатия на кнопку "Сохранить" выполняется Http-запрос:
Запрос от авторизованного пользователя
{{baseUrl}}profile/name/put
{
"name": "Имя", // может быть пустым
"surname": "" // может быть пустым
}
после успешного сохранения система выполняет запрос на получение данных профиля пользователя (profile/info/get) и отображает пользователю обновленные данные на странице "Настройка профиля":
В полях формы ввода отображается имя и фамилия (при наличии)
Обновление фото профиля
{{baseUrl}}profile/image/put
Ограничений к файлу нет
Предлагается использовать минимальное разрешение 200х200 и все форматы фотографий на телефонах
На данный момент реализовано через PickVisualMediaRequest: можно выбрать любые картинки в общих папках, разрешение у пользователя не запрашивается
P.S. т.к. у сервера проблемы с сертификатом для глайда был создан специальный модуль с unsafe клиентом т.к. в конструктор его ничего передавать нельзя, просто взяла код из дата модуля, в один файл какой-то выносить не стала, т.к. всё равно этот класс потом можно будет весь удалить.
Вюшки хидеров: аналитика и трекер и настройки
На данный момент они просто добавлены на макеты, но загрузку данных на всех экранах не реализовала в рамках этой задачи. Данные сохраняются в хранилище при первой загрузке с сервера, чтобы не делать запросы каждый раз. Нужно будет добавить очищение при логауте.