Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Revert "[expo-update][iOS][android] save asset with a key that does n… #13733

Merged
merged 1 commit into from
Jul 22, 2021

Conversation

jkhales
Copy link
Contributor

@jkhales jkhales commented Jul 22, 2021

…ot include an extension. (#12734)"

This reverts commit 63a0bdc.

Why

With the reversion detailed here: #13558 (comment)

We need to also revert this commit which relied on assets to be stored in the format introduced there.

Test Plan

Confirmed publishes of images worked and loaded from local source:

Screen Shot 2021-07-21 at 6 27 56 PM

Checklist

  • Documentation is up to date to reflect these changes (eg: https://docs.expo.io and README.md).
  • This diff will work correctly for expo build (eg: updated @expo/xdl).
  • This diff will work correctly for expo prebuild & EAS Build (eg: updated a module plugin).

@jkhales jkhales force-pushed the @jkhales/revert-ext-drop branch from 045b515 to 05be363 Compare July 22, 2021 01:27
@expo-bot expo-bot added the bot: passed checks ExpoBot has nothing to complain about label Jul 22, 2021
@jkhales jkhales requested a review from esamelson July 22, 2021 01:31
@jkhales jkhales marked this pull request as ready for review July 22, 2021 01:31
Copy link
Contributor

@esamelson esamelson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, let's follow up after this is released to figure out a better solution for the key moving forward.

@jkhales jkhales force-pushed the @jkhales/revert-ext-drop branch from 05be363 to ef59ebc Compare July 22, 2021 01:40
@jkhales jkhales merged commit a5dbb5b into master Jul 22, 2021
@jkhales jkhales deleted the @jkhales/revert-ext-drop branch July 22, 2021 03:07
jkhales pushed a commit that referenced this pull request Jul 22, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bot: passed checks ExpoBot has nothing to complain about
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants