Skip to content

Updated video link in README (marketplace page) #244

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

Merged
Basit-qc merged 4 commits into from
Mar 11, 2025
Merged

Updated video link in README (marketplace page) #244

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
775917e
Modified video link in README
Basit-qc Mar 10, 2025
bf9232f
Updated CHANGELOG
Basit-qc Mar 10, 2025
674c21f
4.10.0
Basit-qc Mar 10, 2025
2180eab
Fixed failing test
Basit-qc Mar 10, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,9 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## [4.10.0] - 2025-03-10
- Modified video link in README

## [4.9.0] - 2025-01-21
- Removed button "View this repo on the Web" from PopUps and Side Panel

Expand Down
2 changes: 1 addition & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ Free for 14 days. Available for JetBrains IDEs and VS Code.
**Visuals**

<video width="600" controls>
<source src="https://codesync-images.s3.amazonaws.com/public/homePage/ai_powered.mp4" type="video/mp4">
<source src="https://dok9jc9tlzc4t.cloudfront.net/public/slack_demo.mp4" type="video/mp4">
Your browser does not support the video tag.
</video>

Expand Down
2 changes: 1 addition & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"displayName": "CodeSync",
"description": "CodeSync's AI-generated summaries provide clear, insightful coding overviews directly in your IDE, keeping your team aligned without manual tracking. Seamlessly integrate with GitHub and Slack to streamline your workflow.",
"icon": "images/icon.png",
"version": "4.9.0",
"version": "4.10.0",
"publisher": "codesync",
"engines": {
"vscode": "^1.86.0",
Expand Down
2 changes: 1 addition & 1 deletion src/codesyncd/websocket/socket_events.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@ export class SocketEvents {
}
if (validTabs.length) {
CodeSyncLogger.debug(`Sending ${validTabs.length} tabs`);
TabsHandler.sendTabsToServer(this.connection, validTabs)
TabsHandler.sendTabsToServer(this.connection, validTabs);
}
CodeSyncState.set(CODESYNC_STATES.BUFFER_HANDLER_RUNNING, false);
}
Expand Down
3 changes: 2 additions & 1 deletion tests/test_utils/auth_utils.test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -132,8 +132,9 @@ describe("logoutHandler", () => {
expect(vscode.commands.executeCommand.mock.calls[0][1]).toStrictEqual("showLogIn");
expect(vscode.commands.executeCommand.mock.calls[0][2]).toBe(true);
await waitFor(2);
expect(vscode.window.showInformationMessage).toHaveBeenCalledTimes(1);
expect(vscode.window.showInformationMessage).toHaveBeenCalledTimes(2);
expect(vscode.window.showInformationMessage.mock.calls[0][0]).toStrictEqual(NOTIFICATION.LOGGED_OUT_SUCCESSFULLY);
Comment on lines -135 to 136
Copy link
Contributor

@ZahidSaeed01 ZahidSaeed01 Mar 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Basit-qc, just for my understanding—why did we update this?

Copy link
Contributor Author

@Basit-qc Basit-qc Mar 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test was failing, not related to this PR. After Logout, we show Welcome Msg to user so there are 2 notifications. Test was asserting 1.

expect(vscode.window.showInformationMessage.mock.calls[1][0]).toStrictEqual(NOTIFICATION.WELCOME_MSG);
});
});

Expand Down
Loading