search

BlackBullSquad / iOS-FakeNFT-StarterProject-Public

1 stars 0 forks source link

Collections sprint 20 #84

Closed maxim-papier closed 1 year ago

maxim-papier commented 1 year ago

Прошлый PR — #62 по нему максимально всё поправил

После ревью (UPDATE)

Исправления по ревью внесены. В том числе интегрирован сервис корзины и сделан свой сервис для получения лайков. Спасибо за ревью @IBDesignable и кросс-ревью @AndreiChenchik. 🤗🤗🤗

https://github.com/BlackBullSquad/iOS-FakeNFT-StarterProject-Public/blob/22a694e5d19121688a8afd1e5629c854b7b024b9/Catalogue-Epic-Plan.md?plain=1#L74-L82

maxim-papier commented 1 year ago

Ой, дал аппрув, а мёржить же только после ревью от ревьювера

Конечно. После того как я однажды уничтожил develop я буду очень аккуратен. :)

maxim-papier commented 1 year ago

Обнаружил глюк вывода ячеек NFT-коллекции при переиспользовании. Проблема возникала из-за спейсера, который был создан в ячейке. Удалил. Компенсировал удаление спейсера в ячейке увеличенным расстоянием между группами в UICollectionView в секции .collection.

Ссылка на коммит: eb156de5079f83a7e01c5ea49e6c109f2b57c288

Screenshot 2023-06-09 at 10 25 28
IBDesignable commented 1 year ago

Прошу прощения за долгое ожидание – проверка занимает больше ожидаемого времени 🙈

IBDesignable commented 1 year ago

Комментарий про растующую память: будет круто если разберешься с кэшем картиночек

Снимок экрана 2023-06-10 в 11 39 26
maxim-papier commented 1 year ago

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

Исправлено https://github.com/BlackBullSquad/iOS-FakeNFT-StarterProject-Public/pull/84/commits/93f8016a6ed423f8fbebeaedbd83f888b6486013

maxim-papier commented 1 year ago

Прошу прощения за долгое ожидание – проверка занимает больше ожидаемого времени 🙈 Всё хорошо. Я понимаю сколько у вас сейчас забот.

maxim-papier commented 1 year ago

Комментарий про растующую память: будет круто если разберешься с кэшем картиночек

Снимок экрана 2023-06-10 в 11 39 26

Я с тревогой смотрел на этот график. Но ведь Kingfisher должен был всё закешировать? Когда второй раз загружаю, то уже видно, что кэш используется. Или тут надо как-то иначе действовать?

maxim-papier commented 1 year ago

@IBDesignable договорились с наставником, что после того как всё будет замерджено другими участиками команды, мне надо будет реализовать взаимодействие с их слоями — с корзиной и избранным в профиле.

maxim-papier commented 1 year ago

Добавлена крутилка при загрузке каталога и коллекции — https://github.com/BlackBullSquad/iOS-FakeNFT-StarterProject-Public/pull/84/commits/eaa80ef1db21e6a79fe8c159fb0dcb85c6677a94

maxim-papier commented 1 year ago

Комментарий про растующую память: будет круто если разберешься с кэшем картиночек

@IBDesignable понял, у Kingfisher гибридный кэш — и диск, и память. Сделал дисковому 512 MB, а тому, что в памяти — 128 MB, но почему-то память по-прежнему уходит 👎 — https://github.com/BlackBullSquad/iOS-FakeNFT-StarterProject-Public/pull/84/commits/dd1076087f39f1995c190ca127e6453e5ce740b7

maxim-papier commented 1 year ago

Исправления по ревью внесены. В том числе интегрирован сервис корзины и сделан свой сервис для получения лайков. Спасибо за ревью @IBDesignable и кросс-ревью @AndreiChenchik

🤗

https://github.com/BlackBullSquad/iOS-FakeNFT-StarterProject-Public/blob/22a694e5d19121688a8afd1e5629c854b7b024b9/Catalogue-Epic-Plan.md?plain=1#L74-L82

IBDesignable commented 1 year ago

Ой, дал аппрув, а мёржить же только после ревью от ревьювера

@AndreiChenchik да можно апрув ставить, но мерджа низя делать) поставь плез сейчас)

maxim-papier commented 1 year ago

Максим Брыков - Ревью диплома 2it.pdf

Получи еще апрув в пулреквесте от Андрея(сейчас реджект стоит)

Работу допускаю до общего мерджа! Спасибо за исправления, ты хорошо постарался!

Обрати внимание: есть еще бажок при работе с загрузкой данных, можно попасть в тупик с белым экраном. Будет круто если разберешься – если нет, ну останется баг в продакшене как в жизни)

Большое спасибо вам. Ваши ревью лучшие из всех, что я встречал за год, как встал на путь разработчика. Скажу более, иногда я что-то делаю и думаю: "А что на это скажет Александра?". 🙃Думаю, что это теперь со мной навсегда.

Буду стараться и впредь! ✌️

maxim-papier commented 1 year ago

https://github.com/BlackBullSquad/iOS-FakeNFT-StarterProject-Public/blob/8f1f47e5bb89f109c30ca18c0dcf62498a8edcba/Catalogue-Epic-Plan.md?plain=1#L80-L88

Влил версию девелопа, которую смерджил @Filosuf, в свою