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

[camera] Fix web imports #26505

Merged
merged 3 commits into from
Jan 19, 2024
Merged

[camera] Fix web imports #26505

merged 3 commits into from
Jan 19, 2024

Conversation

alanjhughes
Copy link
Collaborator

@alanjhughes alanjhughes commented Jan 18, 2024

Why

Closes #26504
Files were renamed to ExpoCamera but the web specific files were still ExponentCamera leading to an error when importing on web.

How

Renamed the files.

Test Plan

Tested in a local project. Web specific files are found and work correctly.

@expo-bot expo-bot added the bot: suggestions ExpoBot has some suggestions label Jan 18, 2024
@expo-bot expo-bot added bot: passed checks ExpoBot has nothing to complain about and removed bot: suggestions ExpoBot has some suggestions labels Jan 18, 2024
@alanjhughes alanjhughes marked this pull request as ready for review January 18, 2024 23:58
@alanjhughes alanjhughes changed the title [camera] Fix web files [camera] Fix web imports Jan 18, 2024
@alanjhughes alanjhughes merged commit 4a40f61 into main Jan 19, 2024
8 checks passed
@alanjhughes alanjhughes deleted the @alanhughes/camera/fix-web-exports branch January 19, 2024 09:52
alanjhughes added a commit that referenced this pull request Jan 22, 2024
@brentvatne brentvatne added the published Changes from the PR have been published to npm label Jan 23, 2024
@WatchmansEase
Copy link

WatchmansEase commented Jan 24, 2024

This bug still exists if you create a new Expo Repository, and setup a fresh component of Expo Camera. It will error on Web. @alanjhughes @brentvatne @lukmccall

@brentvatne
Copy link
Member

@WatchmansEase - can you share a minimal reproducible example repository please? https://stackoverflow.com/help/minimal-reproducible-example

@alanjhughes
Copy link
Collaborator Author

@WatchmansEase - what version of expo-camera are you using? Should be 14.0.2

@alanjhughes
Copy link
Collaborator Author

@WatchmansEase - I've just tested this in a new expo project and it is working on web

@WatchmansEase
Copy link

@alanjhughes My apologies and my thanks - upgrading to expo-camera to 14.0.2 fixed this.

@alanjhughes
Copy link
Collaborator Author

@WatchmansEase - no problem 👍 glad it's working for you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: fingerprint changed bot: passed checks ExpoBot has nothing to complain about published Changes from the PR have been published to npm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[expo-camera] The method or property expo-modules-core.requireNativeViewManager is not available on web
5 participants