-
Notifications
You must be signed in to change notification settings - Fork 3
Add blob upload example #8
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
Conversation
hariria
left a comment
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.
Looks great, some comments
apps/upload-blob/package-lock.json
Outdated
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 think you might have npm installed instead of pnpm
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.
can you remove this file
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.
Not sure if you saw this comment earlier so flagging it here
Co-authored-by: Andrew Hariri <[email protected]>
Co-authored-by: Andrew Hariri <[email protected]>
Co-authored-by: Andrew Hariri <[email protected]>
Co-authored-by: Andrew Hariri <[email protected]>
Co-authored-by: Andrew Hariri <[email protected]>
Co-authored-by: Andrew Hariri <[email protected]>
|
You need to remove the package-lock.json |
hariria
left a comment
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.
Thanks LGTM. I forgot to mention this in the previous pr but would you mind changing the package name of the download example to @shelby-protocol/download-example just to be consistent
No description provided.