deepgram-devs / dg-translation-chrome-ext

A TypeScript chrome extension that uses Deepgram to provide live transcription and translation
MIT License
10 stars 8 forks source link

Issue getting the extension running #10

Closed arndom closed 1 year ago

arndom commented 1 year ago

After following all the steps for getting started, the extension doesn't work, only getting errors:

image image

image

Steps to replicate:

BekahHW commented 1 year ago

Can you tell me about your environment?

Operating System/Version: Language: Browser:

arndom commented 1 year ago

Ok

OS: Windows 11 Language: English Browser: Chrome

On Tue, 11 Oct 2022, 17:55 BekahHW, @.***> wrote:

Can you tell me about your environment?

Operating System/Version: Language: Browser:

— Reply to this email directly, view it on GitHub https://github.com/deepgram-devs/dg-translation-chrome-ext/issues/10#issuecomment-1274997880, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJVPQPWEI3UAYP6GRCHSD6DWCWLYZANCNFSM6AAAAAARCAGGJA . You are receiving this because you authored the thread.Message ID: @.***>

BekahHW commented 1 year ago

Ok, @arndom, can you try to get the latest version of the repo and then run yarn again? I think it was somehow missing the TypeScript devDependency. This should solve it.

arndom commented 1 year ago

ok, @BekahHW, I've pulled the latest updates....it's still showing the error when the pop-up is open:

image image

could it be other extensions affecting it?

BekahHW commented 1 year ago

Just to clarify, you stopped the server, ran yarn, and then ran yarn start?

And your package json file include typescript as a devdependency now right?

arndom commented 1 year ago

Yes, I have done all that and checked that typescript is present:

image
BekahHW commented 1 year ago

Ok, let's try this:

Delete .parcel-cache, dist, node_modules, yarn.lock.

Run yarn and then yarn start.

Hopefully, this fixes the problem.

Just double checking, you don't have a package-lock.json file, right? If you do, delete that as well.

arndom commented 1 year ago

@BekahHW, so...I've tried it but still get the same error.

Also, I've tried loading the unpacked extension from dist instead of src and have been getting:

image
BekahHW commented 1 year ago

I can't do it today, but do you want to do a video call in the next couple of days and see if we can get it working together?

arndom commented 1 year ago

Sure, that would be really helpful...when would you be available?

BekahHW commented 1 year ago

@arndom I think I figured it out. I missed a step in the readme. I have it all updated now, but here are a couple of updates. Notice that we're loading from the dist folder now.

arndom commented 1 year ago

@BekahHW I tried it; here's my result:

image

arndom commented 1 year ago

@BekahHW I got it past the previous error by editing the generated popupjs in the dist folder to use the generated content-script.....it asks to choose a tab to share audio but I cant seem to find the transcription of the playing video.

muckitymuck commented 1 year ago

This is the error I am getting trying to get this to work: image

arndom commented 1 year ago

@BekahHW I got the error not to come up consistently by changing the way the popup.ts reads the content-script, issue now is that it seems to be sharing my audio, but I can't see any transcription.

image

popup.ts:

image

content-script.ts:

image

popup.html:

image
BekahHW commented 1 year ago

That's great news, @arndom. Do you want to create a PR for this and I'll merge in the fix?

arndom commented 1 year ago

@BekahHW, Here's the PR: https://github.com/deepgram-devs/dg-translation-chrome-ext/pull/18

BekahHW commented 1 year ago

@arndom did you delete .parcel-cache, dist, node_modules, yarn.lock, and then run yarn, yarn build, and then yarn start? I'm getting a different error

image
arndom commented 1 year ago

Yes I did all that, that error seems to happen when the manage extension tab is open: chrome://extensions/, I closed the tab and reloaded the extension to get it working

BekahHW commented 1 year ago

@arndom it's interesting bc I can console.log the transcript in showLatestTranscript() and it shows up in my console, but I'm not getting it in the popup.html. Feels like we're getting closer though

BekahHW commented 1 year ago

@arndom I just pushed up a fix. Should be working now!

arndom commented 1 year ago

awesome, I'll pull and try

arndom commented 1 year ago

@BekahHW It's working

CrazyDanceFunnyDanceGIF

BekahHW commented 1 year ago

@arndom high five gif