LBRYFoundation/Watch-on-LBRY
1
0
Fork 0
mirror of https://github.com/LBRYFoundation/Watch-on-LBRY.git synced 2025-08-23 17:47:26 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #99 from DeepDoge/master

Browse source 
On error background returns undefined but it should throw
This commit is contained in:
kodxana 2022-01-25 17:07:39 +01:00 • committed by GitHub
commit e501a8b828
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/scripts/background.ts
View file

@ -17,7 +17,7 @@ async function lbryPathnameFromVideoId(videoId: string): Promise<string | void>
}
chrome.runtime.onMessage.addListener(({ videoId }: { videoId: string }, sender, sendResponse) => {
lbryPathnameFromVideoId(videoId).then((lbryPathname) => sendResponse(lbryPathname))
lbryPathnameFromVideoId(videoId).then((lbryPathname) => sendResponse(lbryPathname)).catch((err) => sendResponse(err))
return true
})

4
src/scripts/ytContent.tsx
View file

@ -100,7 +100,9 @@ async function findVideoElement() {
// We should get this from background, so the caching works and we don't get errors in the future if yt decides to impliment CORS
async function requestLbryPathname(videoId: string) {
return await new Promise<string | null>((resolve) => chrome.runtime.sendMessage({ videoId }, resolve))
const response = await new Promise<string | null | Error>((resolve) => chrome.runtime.sendMessage({ videoId }, resolve))
if (response instanceof Error) throw response
return response
}
// Start