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
feat: add extension example #10
Conversation
262ea1c
to
751be07
Compare
64540ef
to
3bbebbe
Compare
{ | ||
"private": true, | ||
"name": "@wallet-standard/parcel-packager-webextension", | ||
"version": "0.1.0", | ||
"author": "Exodus Maintainers <maintainers@exodus.io>", | ||
"repository": "https://github.com/wallet-standard/wallet-standard", | ||
"license": "MIT", | ||
"source": "src/WebExtensionPackager.js", | ||
"main": "lib/WebExtensionPackager.js", | ||
"engines": { | ||
"node": ">=12.0.0", | ||
"parcel": "^2.7.0" | ||
}, | ||
"scripts": { | ||
"clean": "shx mkdir -p .parcel-cache dist lib && shx rm -rf .parcel-cache dist lib", | ||
"build": "parcel build" | ||
}, | ||
"dependencies": { | ||
"@parcel/plugin": "2.7.0", | ||
"@parcel/utils": "2.7.0", | ||
"nullthrows": "^1.1.1", | ||
"parcel": "^2.3.2" | ||
} | ||
} |
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 will live here temporarily until parcel-bundler/parcel#8424 is merged or published separately.
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.
Very nice, this implementation looks super clean and well organized.
@@ -28,6 +28,7 @@ | |||
"publish:alpha": "pnpm -r publish --tag alpha" | |||
}, | |||
"devDependencies": { | |||
"@types/chrome": "^0.0.195", |
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 dep should probably be specific to the extension package(s) right?
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.
I tried that first, but it didn't recognize the chrome
global that way. Maybe additional TS config is needed at the extension
example package?
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.
Happy to fix in a follow-up PR.
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.
Cool, thanks!
feat: add dApp example
Summary
This PR adds a browser extension that uses the standard natively to the examples.
For now, it includes: