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.

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

feat(participant): remove in-app notification for participant VSCODE-662 #899

Merged
merged 63 commits into from
Dec 9, 2024

Conversation

gagik
Copy link
Contributor

@gagik gagik commented Dec 6, 2024

We decided to remove the notification for Copilot extension so we don't spam existing users.

This essentially reverts changes in #875

@gagik gagik changed the base branch from main to gagik/better-tracking December 6, 2024 09:19
@gagik gagik force-pushed the gagik/remove-notification branch from 746858b to 576a11f Compare December 6, 2024 09:49
@gagik gagik force-pushed the gagik/remove-notification branch from 576a11f to a9a497f Compare December 6, 2024 10:01
@gagik gagik requested a review from alenakhineika December 6, 2024 10:12
@gagik gagik force-pushed the gagik/better-tracking branch from 647ced6 to 15cdbb5 Compare December 6, 2024 10:18
@gagik gagik force-pushed the gagik/better-tracking branch from 15cdbb5 to 32a4c52 Compare December 6, 2024 10:28
@gagik gagik force-pushed the gagik/remove-notification branch from a9a497f to 857c8c0 Compare December 6, 2024 12:26
@gagik gagik force-pushed the gagik/remove-notification branch from 841864b to 857c8c0 Compare December 6, 2024 13:34
Base automatically changed from gagik/better-tracking to main December 9, 2024 14:56
@gagik gagik merged commit 6af38f8 into main Dec 9, 2024
6 checks passed
@gagik gagik deleted the gagik/remove-notification branch December 9, 2024 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants