-
Notifications
You must be signed in to change notification settings - Fork 102
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
[DO NOT MERGE ] chore - completed migration to react #113
base: main
Are you sure you want to change the base?
Conversation
* Added sidepanel * Updated
@arsenkylyshbek , can you please do a review. Need to merge in 2 hours. K thx bye! |
@sansyrox mmm +9,000 lines of code - would love to go through every single one of them!! busy shipping new features tho, sorry 😘 |
|
I will merge it in 2 hours. |
@yogesh895 can you please do a PR review or have a look? |
Happy to also look! Love the banter on this chat😜 |
@shivaylamba , please do man 😄 |
@@ -453,10 +474,10 @@ function downloadTranscript() { | |||
|
|||
console.log("Regular transcript:", textContent); |
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.
This should be now
console.log("Google Meeet transcript:", textContent); ?
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.
|
need to test realtime suggestions and css