Skip to content

Fix assignment event time inconsistency with GitHub.com#7061

Merged
alexr00 merged 3 commits intomainfrom
copilot/fix-7007
Jun 25, 2025
Merged

Fix assignment event time inconsistency with GitHub.com#7061
alexr00 merged 3 commits intomainfrom
copilot/fix-7007

Conversation

Copy link
Contributor

Copilot AI commented Jun 23, 2025

When multiple assignment/unassignment events happen within 10 minutes by the same actor, VSCode consolidates them into a single timeline item. However, VSCode was showing the most recent time while GitHub.com shows the earliest time, causing confusion for users.

Before:

  • VSCode: Shows "12 minutes ago" (most recent assignment time)
  • GitHub.com: Shows "14 minutes ago" (earliest assignment time)

After:

  • Both VSCode and GitHub.com now consistently show "14 minutes ago" (earliest assignment time)

The fix removes the line that was updating the consolidated event's timestamp to the newest time, instead keeping the original (earliest) timestamp to match GitHub.com's behavior.

Technical Details:
In webviews/components/timeline.tsx, when consolidating assignment events that occur within 10 minutes by the same actor, the code was setting lastEvent.createdAt = newEvent.createdAt which used the most recent time. This line has been removed so the earliest time is preserved.

Fixes #7007.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>
Copilot AI changed the title [WIP] Times are inconsistent with .com Fix assignment event time inconsistency with GitHub.com Jun 23, 2025
Copilot AI requested a review from alexr00 June 23, 2025 15:09
Copy link
Member

@alexr00 alexr00 left a comment

Choose a reason for hiding this comment

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

@copilot, please remove the comment

Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jun 24, 2025

@copilot, please remove the comment

Done. Removed the comment in 3df9ee2.

Copilot AI requested a review from alexr00 June 24, 2025 09:51
@alexr00 alexr00 marked this pull request as ready for review June 25, 2025 13:39
@alexr00 alexr00 enabled auto-merge (squash) June 25, 2025 13:39
@vs-code-engineering vs-code-engineering bot added this to the June 2025 milestone Jun 25, 2025
@alexr00 alexr00 merged commit 57b2e89 into main Jun 25, 2025
3 checks passed
@alexr00 alexr00 deleted the copilot/fix-7007 branch June 25, 2025 17:16
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.

Times are inconsistent with .com

4 participants