Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[storage-resize-images] Always use posix paths when writing to storage #1416

Merged
merged 1 commit into from
Feb 1, 2023

Conversation

joehan
Copy link
Collaborator

@joehan joehan commented Jan 23, 2023

Switch storage-resize-images to always use POSIX paths when writing to Storage.

This addresses an issue (#1732) where, when emulating this extension on a Windows machine, resized images would be written to the root of the bucket with the Windows path as the filename.

Fixes #1732.

@simenness
Copy link

👋🏻😅

It's been a week.

@joehan joehan requested a review from dackers86 January 30, 2023 18:49
@dackers86 dackers86 self-assigned this Jan 31, 2023
@dackers86 dackers86 added this to Under consideration in Extension Update Tracker via automation Feb 1, 2023
@dackers86 dackers86 moved this from Under consideration to In review in Extension Update Tracker Feb 1, 2023
@dackers86 dackers86 added the in-review Awaiting review by FE team. label Feb 1, 2023
Extension Update Tracker automation moved this from In review to Launch approval pending Feb 1, 2023
@dackers86 dackers86 merged commit 3c8bb6d into next Feb 1, 2023
Extension Update Tracker automation moved this from Launch approval pending to Launched Feb 1, 2023
@dackers86 dackers86 removed the in-review Awaiting review by FE team. label Feb 1, 2023
@cabljac cabljac deleted the joehanley/storage-resize-images-emalator-windows branch May 25, 2023 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

Extension 'firebase/storage-resize-images' emulator, not working well with Windows
3 participants