Skip to content

Comments

Improve tooltip status text for product availability issues#6293

Merged
mirekm merged 2 commits intomainfrom
fix/channel-availabilit-dot-tooltip
Jan 29, 2026
Merged

Improve tooltip status text for product availability issues#6293
mirekm merged 2 commits intomainfrom
fix/channel-availabilit-dot-tooltip

Conversation

@mirekm
Copy link
Member

@mirekm mirekm commented Jan 28, 2026

No description provided.

@mirekm mirekm requested a review from a team as a code owner January 28, 2026 21:55
@mirekm mirekm added skip changeset Use if your changes doesn't need entry in changelog test deployment Deploy Pull Request to *.saleor.rocks environment labels Jan 28, 2026
Copilot AI review requested due to automatic review settings January 28, 2026 21:55
@changeset-bot
Copy link

changeset-bot bot commented Jan 28, 2026

⚠️ No Changeset found

Latest commit: 3417451

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
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 improves the tooltip status text displayed for products with availability issues. When a product has availability issues in a channel, the tooltip now shows "Issues" / "Has availability issues" as the primary status instead of the underlying publication status (live/scheduled/hidden).

Changes:

  • Added internationalization messages for issues status and description
  • Modified status label and description logic to prioritize showing "Issues" when availability issues are present
  • Updated locale file with new message translations

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/products/components/ProductDoctor/messages.ts Added status_issues and statusDescription_issues message definitions following the established naming convention
src/products/components/ProductDoctor/AvailabilityChannelItem.tsx Modified getStatusLabel() and getStatusDescription() to return issues-related messages when hasIssues is true, prioritizing issue visibility over publication status
locale/defaultMessages.json Added translations for the two new message IDs (PF7Zsp and vsSdHA) with appropriate context

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov
Copy link

codecov bot commented Jan 28, 2026

Codecov Report

❌ Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 42.49%. Comparing base (972ca64) to head (3417451).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...mponents/ProductDoctor/AvailabilityChannelItem.tsx 0.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6293      +/-   ##
==========================================
- Coverage   42.50%   42.49%   -0.01%     
==========================================
  Files        2492     2492              
  Lines       43216    43220       +4     
  Branches    10162     9779     -383     
==========================================
  Hits        18368    18368              
- Misses      24812    24816       +4     
  Partials       36       36              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mirekm mirekm merged commit 9aa74cd into main Jan 29, 2026
13 of 14 checks passed
@mirekm mirekm deleted the fix/channel-availabilit-dot-tooltip branch January 29, 2026 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip changeset Use if your changes doesn't need entry in changelog test deployment Deploy Pull Request to *.saleor.rocks environment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants