-
Notifications
You must be signed in to change notification settings - Fork 893
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
WebTorrent: Add "Save All Files..." feature #3146
Conversation
2017846
to
e515586
Compare
"@types/parse-torrent": "^5.8.3", | ||
"@types/webtorrent": "^0.98.5", | ||
"bignumber.js": "^7.2.1", | ||
"bluebird": "^3.5.1", | ||
"clipboard-copy": "^2.0.0", | ||
"jszip": "^3.2.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.
cc @diracdeltas for adding a new dep here.
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.
++
@@ -7,7 +7,7 @@ | |||
"scripts": ["extension/out/brave_webtorrent_background.bundle.js"], | |||
"persistent": true | |||
}, | |||
"permissions": ["dns", "tabs", "windows", "<all_urls>"], | |||
"permissions": ["downloads", "dns", "tabs", "windows", "<all_urls>"], |
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.
cc @diracdeltas for granting a new permission to webtorrent extension.
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.
seems ok
Currently there is no ability to save all files at once. User has to save each file separately. This commit fixes that. Fixes: brave/brave-browser#1230
e515586
to
31af2e6
Compare
Force pushed with the stray |
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.
Codes LGTM and test plan works fine.
I'm going to merge it here without waiting for CI again since the latest force push only had a |
fix brave/brave-browser#5662 storybook went broken after #3146. this PR fixes it by updating webtorrent's props in storybook
Currently there is no ability to save all files at once. The user has to save each file separately. This PR fixes that.
Fixes: brave/brave-browser#1230
Submitter Checklist:
npm run lint
)git rebase master
(if needed).git rebase -i
to squash commits (if needed).Test Plan:
Reviewer Checklist:
After-merge Checklist:
changes has landed on.