Skip to content

[pull] main from expo:main#23

Merged
pull[bot] merged 7 commits into
code:mainfrom
expo:main
Jul 21, 2025
Merged

[pull] main from expo:main#23
pull[bot] merged 7 commits into
code:mainfrom
expo:main

Conversation

@pull

@pull pull Bot commented Jul 21, 2025

Copy link
Copy Markdown

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.3)

Can you help keep this open source service alive? 💖 Please sponsor : )

lukmccall and others added 7 commits July 21, 2025 12:38
# Why

The modal demo was messy

# How

<!--
How did you build this feature or fix this bug and why?
-->

# Test Plan

<!--
Please describe how you tested this change and how a reviewer could
reproduce your test, especially if this PR does not include automated
tests! If possible, please also provide terminal output and/or
screenshots demonstrating your test/reproduction.
-->

# Checklist

<!--
Please check the appropriate items below if they apply to your diff.
-->

- [ ] I added a `changelog.md` entry and rebuilt the package sources
according to [this short
guide](https://github.com/expo/expo/blob/main/CONTRIBUTING.md#-before-submitting)
- [ ] This diff will work correctly for `npx expo prebuild` & EAS Build
(eg: updated a module plugin).
- [ ] Conforms with the [Documentation Writing Style
Guide](https://github.com/expo/expo/blob/main/guides/Expo%20Documentation%20Writing%20Style%20Guide.md)
# Why

When loading image from PH with target size like 50x50 it's being cached
as the original image, later, when we want to load the same image with
bigger size, like 1000x1000 its being picked from cache ignoring that
it's bigger size now.

# How

Images from PH are already loaded with `targetSize` so we dont want to
store them as "original" image, instead we store them only as a
processed image with size included.
@pull pull Bot locked and limited conversation to collaborators Jul 21, 2025
@pull pull Bot added the ⤵️ pull label Jul 21, 2025
@pull pull Bot merged commit c9fa0aa into code:main Jul 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants