Изменение фото профиля и большой рефактор сетевого слоя / defaults #268
Conversation
This file contains hidden or 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
* Починил краш при логировании JSON Был такой краш: *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '*** +[NSJSONSerialization dataWithJSONObject:options:error:]: Invalid top-level type in JSON write' * Добавил Content-Length для запросов с multipart * Рефактор BodyMaker * Убрал лишний модификатор доступа public * Добавил отправку фотографии в запрос `editUser` И доработал конвертацию картинки перед отправкой на сервер * В процессе - Сделал отдельный экран для основного пользователя и отдельный для всех остальных - Поправил обновление аватарки при помощи модификатора `.id` * В процессе - Вынес `defaults` из запроса для авторизации в экран - Добавил анимации на экран профиля при появлении соц.кнопок - Переделываю некоторые свойства `defaults` на вычисляемые * В процессе - Поправил авторизацию - Дорабатываю сетевой слой * Убрал главного пользователя из экрана UserDetails * navigationBar -> topBar Апи обновился уже давно * Мелкий рефактор * Добавил ошибку и текст для 401 в клиенте * Рефактор и мелкие правки - Не показываем кнопку обновления на экране со списком дневников для iOS 16+ - На экране со списком заблокированных пользователей загружаем данные снова при рефреше - Убрал несколько лишних тогглов для отображения алерта с ошибкой - можно опираться только на наличие текста ошибки * MVP: выбор нового аватара * Мелкий рефактор * Локализация для InfoPlist * Рефактор соц.фичей Сохраняем заявки в друзья, друзей и черный список в тех местах приложения, где эти запросы вызываются, а не в сетевом клиенте
* Добавил локальный пакет SWAlert * Показываю все алерты через синглтон Дополнительно доработал получение статуса в запросах, где не возвращаются данные: если пришел ответ с ошибкой, то передаем ее наверх, иначе возвращаем `false` * Доработал локализацию в алертах Дополнительно добавил локализацию на случай добавления/удаления пользователя из черного списка * Поднял версию сборки
- Переименовал метод презентации алерта - Добавил комментарии - Убрал `nonisolated` - Применил на экране авторизации
Возвращаем обычную ошибку 401 вместо `ClientError.forceLogout`
- Вынес из сетевого слоя все упоминания `defaults` кроме токена и логаута - Актуализировал неиспользуемые запросы для регистрации и удаления пользователя - Доработал логику добавления в друзья, обработки заявки на добавление в друзья
easydev991
added
улучшение
easydev991
changed the title
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.
Заметное для пользователя:
Рефактор:
defaultsкроме токена и логаутаSWAlert)