Skip to content

fix: use require in ReactNativeDriver#6814

Merged
pleerock merged 1 commit intotypeorm:masterfrom
imnotjames:fix-react-native
Oct 1, 2020
Merged

fix: use require in ReactNativeDriver#6814
pleerock merged 1 commit intotypeorm:masterfrom
imnotjames:fix-react-native

Conversation

@imnotjames
Copy link
Contributor

Because react native uses the browser context we can't use PlatformTools.load so this reverts the behavior for this file to use a standard require call

Fixes #6811

Because react native uses the browser context we can't use `PlatformTools.load` so this reverts the behavior for this file to use a standard `require` call

Fixes typeorm#6811
Copy link

@oleksandr-dziuban oleksandr-dziuban left a comment

Choose a reason for hiding this comment

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

It's working perfectly now, double checked. Thanks a lot!

@pleerock pleerock merged commit 1a6383c into typeorm:master Oct 1, 2020
@pleerock
Copy link
Member

pleerock commented Oct 1, 2020

Thank you @imnotjames !

@imnotjames imnotjames added the hacktoberfest-accepted label hacktoberfest label Oct 7, 2020
zaro pushed a commit to zaro/typeorm that referenced this pull request Jan 12, 2021
Because react native uses the browser context we can't use `PlatformTools.load` so this reverts the behavior for this file to use a standard `require` call

Fixes typeorm#6811
@imnotjames imnotjames deleted the fix-react-native branch June 21, 2021 00:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest-accepted label hacktoberfest

Projects

None yet

3 participants