Skip to content

marketing: remove product updates API hack - #6262

Open
wesrupert wants to merge 1 commit into
masterfrom
feat/wr/marketing/dev
Open

wesrupert wants to merge 1 commit into
masterfrom
feat/wr/marketing/dev

Conversation

@wesrupert

Copy link
Copy Markdown
Contributor

BE has resolved the underlying issue, remove product updates hack

Copilot AI balanced review requested due to automatic review settings October 2, 2026 17:07

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The focused cleanup matches the updated backend response contract without leaving related references.

Review effort: Balanced
Findings: None

What changed in this PR

Removes the obsolete product-updates API response workaround after the backend fix.

Changes:

  • Stops converting non-array updates responses with Object.values.
File Description
app/​services/​announcements.ts Removes obsolete sparse-array normalization.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@bundlemon

bundlemon Bot commented Oct 2, 2026

Copy link
Copy Markdown

BundleMon

Files updated (1)
Status Path Size Limits
✅ renderer.(hash).js
10.66MB (-87B 0%) -
Unchanged files (3)
Status Path Size Limits
✅ vendors~renderer.(hash).js
4.67MB -
✅ updater.js
115.29KB -
✅ guest-api.js
40.23KB -

Total files change -87B 0%

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

@michelinewu michelinewu added the tech debt infra changes, refactors, etc label Oct 2, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tech debt infra changes, refactors, etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants