-
Notifications
You must be signed in to change notification settings - Fork 33
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
Upgrade to newest RN, add Expo SDK, general cleanup. #550
Changes from 1 commit
19803b4
7389711
306696a
1510c54
4da703e
b659ad9
67d40b6
400006d
de2d89d
c9fa72c
6f2a6ac
02f5a64
c299221
81e5b70
da2959e
5b61843
6f9f465
db216d1
5fa7faf
ee50410
0763ca2
8c40151
a59412b
b73129c
e0e1d43
3d1fedc
eab42c7
fc98b53
b925b0f
e5d25cd
8ef3f8a
6104b05
ffc1fe9
473e311
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -30,6 +30,9 @@ | |
} | ||
} | ||
}, | ||
"overrides": { | ||
"@digitalbazaar/http-client": "4.1.0" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. there were 2 versions of this installed, because @interop/did-web-resolver also depends on a different version of http-client |
||
}, | ||
"dependencies": { | ||
"@digitalcredentials/cborld": "^4.3.4", | ||
"@digitalcredentials/did-method-key": "^2.0.3", | ||
|
@@ -54,6 +57,7 @@ | |
"big-integer": "^1.6.49", | ||
"bson": "4.4.1", | ||
"expo": "~50.0.6", | ||
"expo-crypto": "~12.8.0", | ||
"expo-dev-client": "~3.3.8", | ||
"expo-secure-store": "~12.8.1", | ||
"expo-splash-screen": "~0.26.4", | ||
|
@@ -104,8 +108,7 @@ | |
"text-encoding": "^0.7.0", | ||
"util": "^0.10.4", | ||
"validator": "^13.11.0", | ||
"whatwg-fetch": "^3.6.20", | ||
"expo-crypto": "~12.8.0" | ||
"whatwg-fetch": "^3.6.20" | ||
}, | ||
"devDependencies": { | ||
"@react-native/eslint-config": "^0.73.2", | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,25 @@ | ||
diff --git a/node_modules/@digitalbazaar/http-client/package.json b/node_modules/@digitalbazaar/http-client/package.json | ||
index 4dd70e5..5a24632 100644 | ||
--- a/node_modules/@digitalbazaar/http-client/package.json | ||
+++ b/node_modules/@digitalbazaar/http-client/package.json | ||
@@ -5,16 +5,14 @@ | ||
"license": "BSD-3-Clause", | ||
"type": "module", | ||
"main": "./dist/cjs/index.cjs", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. metro loaded this implementation, which is for node. That was the issue |
||
- "exports": { | ||
- "require": "./dist/cjs/index.cjs", | ||
- "import": "./lib/index.js" | ||
- }, | ||
"browser": { | ||
"./lib/agentCompatibility.js": "./lib/agentCompatibility-browser.js", | ||
"./tests/utils.cjs": "./tests/utils-browser.cjs" | ||
}, | ||
- "react-native": { | ||
- "./lib/agentCompatibility.js": "./lib/agentCompatibility-browser.js" | ||
+ "exports": { | ||
+ "react-native": "./lib/index.js", | ||
+ "require": "./dist/cjs/index.cjs", | ||
+ "import": "./lib/index.js" | ||
}, | ||
"scripts": { | ||
"rollup": "rollup -c rollup.config.js", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is a major change in how metro resolves packages. It's not needed but it's going to be the default in future, so I figured I'd give it a try