Skip to content

Implement migration for prerelease version handling in notifications#610

Merged
tnaum-ms merged 3 commits into
nextfrom
dev/tnaum/what-is-new-optimization
Apr 24, 2026
Merged

Implement migration for prerelease version handling in notifications#610
tnaum-ms merged 3 commits into
nextfrom
dev/tnaum/what-is-new-optimization

Conversation

@tnaum-ms
Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings April 24, 2026 10:18
@tnaum-ms tnaum-ms requested a review from a team as a code owner April 24, 2026 10:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the release notes notification service to avoid prerelease builds “consuming” the stable release notification, and adds a one-time storage migration for users impacted by a prior 0.8.0-bugbash prerelease.

Changes:

  • Skip showing release notes notifications (and skip updating stored state) when the current extension version is a prerelease (e.g., -beta, -preview, -bugbash).
  • Add a one-time globalState migration to fix stored version values affected by a 0.8.0-bugbash prerelease.
  • Update Jest tests to reflect the new prerelease behavior.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File Description
src/services/releaseNotesNotification.ts Adds prerelease detection/short-circuit and a one-time migration to correct stored version state.
src/services/releaseNotesNotification.test.ts Updates/expands unit tests for prerelease and migration-related behavior.

Comment thread src/services/releaseNotesNotification.ts
Comment thread src/services/releaseNotesNotification.ts
Comment thread src/services/releaseNotesNotification.test.ts
Comment thread src/services/releaseNotesNotification.test.ts
Comment thread src/services/releaseNotesNotification.test.ts
@github-actions
Copy link
Copy Markdown
Contributor

✅ Code Quality Checks

Check Status How to fix
Localization (l10n) ✅ Passed
ESLint ✅ Passed
Prettier formatting ✅ Passed

This comment is updated automatically on each push.

@github-actions
Copy link
Copy Markdown
Contributor

📦 Build Size Report

Metric Base (next) PR Delta
VSIX (vscode-documentdb-0.8.0-beta.vsix) 6.62 MB 6.62 MB ⬆️ +0 KB (+0.0%)
Webview bundle (views.js) 5.78 MB 5.78 MB ✅ 0 KB (0.0%)

Download artifact · updated automatically on each push.

@tnaum-ms tnaum-ms merged commit 919649c into next Apr 24, 2026
8 checks passed
@tnaum-ms tnaum-ms deleted the dev/tnaum/what-is-new-optimization branch April 24, 2026 12:18
@tnaum-ms tnaum-ms mentioned this pull request May 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

4 participants