Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Creating worker instance and getting or creating a new sharedworkspace for co-presence #745

Merged
merged 36 commits into from
Nov 20, 2023
Merged

Creating worker instance and getting or creating a new sharedworkspace for co-presence #745

merged 36 commits into from
Nov 20, 2023

Conversation

ritikramuka
Copy link
Contributor

@ritikramuka ritikramuka commented Oct 19, 2023 •
edited
Loading

Based on #717

@ritikramuka ritikramuka requested a review from a team as a code owner October 19, 2023 05:21
@ritikramuka ritikramuka changed the title passed entity id while loading container & few nit changes Creating worker instance and getting or creating a new sharedworkspace for co-presence Oct 19, 2023
Base automatically changed from users/ramukaritik/compile-copresence-worker-file to main October 24, 2023 18:00
tyaginidhi
tyaginidhi requested changes Oct 26, 2023
View reviewed changes
src/web/client/WebExtensionContext.ts Outdated Show resolved Hide resolved
src/web/client/WebExtensionContext.ts Outdated Show resolved Hide resolved
src/web/client/extension.ts Show resolved Hide resolved
src/web/client/extension.ts Outdated Show resolved Hide resolved
src/web/client/extension.ts Outdated Show resolved Hide resolved
src/web/client/extension.ts Outdated Show resolved Hide resolved
src/web/client/extension.ts Outdated Show resolved Hide resolved
src/web/client/extension.ts Outdated Show resolved Hide resolved
ritikramuka and others added 15 commits October 26, 2023 18:32
@ritikramuka
moved co-presence on vscode for web related parameters to a single place
43f883c
@ritikramuka
Merge branch 'main' into users/ramukaritik/creating-webworker-instance
ad6d855
@ritikramuka
disabled co-presence feature for users as under development
a2922d0
@ritikramuka
update in way of joining path
64f0fa3
@ritikramuka
added const for worker event messages
618928a
@ritikramuka
renamed a worker event mesages
1fcfe2e
@ritikramuka
Merge branch 'main' into users/ramukaritik/creating-webworker-instance
1bcf192
@ritikramuka
remove redundant event registery of onDidChangeTabs
6df6477
@ritikramuka
Merge branch 'users/ramukaritik/creating-webworker-instance' of https…
0555759 
…://github.com/microsoft/powerplatform-vscode into users/ramukaritik/creating-webworker-instance
@ritikramuka
moved populateSharedWorkSpace to authenticateAndUpdateDataverseProper…
474ee5d 
…ties as no need to call it again and again with file save, and added concurreny handler to api call and telemetry to track logs
@ritikramuka
disabling co-presence feature hard coded
b4e9319
@ritikramuka
added worker in webextension context
b2568e6
@ritikramuka
used concurrency handler to make fetch call worker script
5d9d994
@ritikramuka
eslint error fixed
eacad21
@ritikramuka
Merge branch 'main' into users/ramukaritik/creating-webworker-instance
1e6e2bf
@ritikramuka ritikramuka enabled auto-merge (squash) October 30, 2023 20:48
@ritikramuka ritikramuka requested a review from petrochuk October 30, 2023 21:04
@ritikramuka ritikramuka disabled auto-merge October 31, 2023 13:48
@tyaginidhi tyaginidhi enabled auto-merge (squash) November 8, 2023 06:32
@tyaginidhi tyaginidhi merged commit 9980242 into main Nov 20, 2023
6 checks passed
@tyaginidhi tyaginidhi deleted the users/ramukaritik/creating-webworker-instance branch November 20, 2023 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tyaginidhi tyaginidhi tyaginidhi approved these changes

@BidishaMS BidishaMS BidishaMS approved these changes

@amitjoshi438 amitjoshi438 amitjoshi438 approved these changes

@tehcrashxor tehcrashxor tehcrashxor approved these changes

@ashishchoudhary001 ashishchoudhary001 ashishchoudhary001 approved these changes

@petrochuk petrochuk Awaiting requested review from petrochuk

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@ritikramuka @ashishchoudhary001 @tehcrashxor @tyaginidhi @petrochuk @amitjoshi438 @BidishaMS