search

mindbox-cloud / ios-sdk

Other
6 stars 16 forks source link

MBX-3507: New image size for rich push notifications #423

Closed justSmK closed 4 months ago

justSmK commented 4 months ago

[iOS] Ресайз картинок в пушах#3507

Не стал выносить расчет imageRatio в расширение какое-нибудь или выставлять размеры ImageView вообще через intrinsicContentSize. Посчитал, что здесь это будет перебор и нужно оставить как проще и как уже было проверено.

justSmK commented 4 months ago

Не мержить в develop. Можно собрать сборку отдельно, чтобы можно было протестировать, но мержить до полной готовности не будем.

Из-за stopAccessingSecurityScopedResource или в целом из-за картинок в пушах?

Vailence commented 4 months ago

Не мержить в develop. Можно собрать сборку отдельно, чтобы можно было протестировать, но мержить до полной готовности не будем.

Из-за stopAccessingSecurityScopedResource или в целом из-за картинок в пушах?

Из-за того что релиз должен быть одновременно +- с фронтом

justSmK commented 4 months ago

Не мержить в develop. Можно собрать сборку отдельно, чтобы можно было протестировать, но мержить до полной готовности не будем.

Из-за stopAccessingSecurityScopedResource или в целом из-за картинок в пушах?

Из-за того что релиз должен быть одновременно +- с фронтом

Так у нас и так релиз следующий это как раз resize images. Плюс RC сборку же мы будем релизить уже на следующей неделе и она должна идти через develop, разве нет?