generated from deepgram-devs/oss-repo-template
-
Notifications
You must be signed in to change notification settings - Fork 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
Add TypeScript support for tab #12
Comments
Can I take this one? |
@JesseRWeigel all you! |
Hey there, I'd like to work on this issue! |
I haven't done TS much, but I see tabId and tab.id are initially mentioned on line 16. Am I going to be doing something similar to what you've got on line 60, |
Hey @AlexVCS when you hover over tab.id, what information are you seeing about why TS isn't ok with it? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Proposed changes
In
popup.ts
we're getting some warnings fortabId
andtab.id
. Update to use types and eliminate the warnings.The text was updated successfully, but these errors were encountered: