Skip to content

Update/layout - #45

Merged
laithanhhung merged 2 commits into
devfrom
update/layout
Aug 16, 2026
Merged

laithanhhung merged 2 commits into
devfrom
update/layout

Conversation

@laithanhhung

Copy link
Copy Markdown
Member

No description provided.

laithanhhung and others added 2 commits August 16, 2026 21:14
The pricing tab saved a row at a time, from a check button per row. Editing a
market's prices meant one write per product, each with its own success toast,
and no way to look over the set before it went out.

Rows are now edited in place and nothing is written until the toolbar says so.
What is unsaved is derived from the drafts rather than tracked by hand, so only
fields that actually differ are sent: a price typed back to its original costs
no request, and a tag set merely reselected is not replaced. Drafts are keyed by
product and held for the whole inventory, so an edit survives paging, sorting
and filtering.

Prices and quantities are checked against the same rules the backend enforces
before the batch leaves, because half a saved batch is the state nobody can
read. Failures are reported per listing and the rows that failed keep their
edits through the refresh that follows.

Also adds `pricing.tagsTooMany`, which the code has asked for since the tag
catalog landed and neither translation file carried.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
The player chose the native-HLS path on `canPlayType('application/vnd.apple.
mpegurl')` alone. Chrome 151 answers "maybe" to that and then cannot load the
manifest, so the `.m3u8` went straight onto the element, failed with
NotSupportedError, and the hero dropped the player — leaving the built scene.
hls.js was never imported and the MP4 was never tried, on a browser whose MSE
would have streamed it without complaint.

Native HLS is now reserved for browsers with no MSE at all, which is iOS Safari
and the only place it was ever meant for. An element error also steps down to
the MP4 before the player gives up, so a browser that misreports its support
again costs a rendition rather than the whole hero.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@kody-ai

kody-ai Bot commented Aug 16, 2026 •

Copy link
Copy Markdown

Code Review Could Not Complete ⚠️

The review failed before suggestions could be generated.

Reason: The configured API key (google_gemini) is out of credits or has hit its billing limit. Top up the account or adjust the plan.

After fixing the issue, comment @kody review on this PR to re-run the review.

Kody Guide: Usage and Configuration
Interacting with Kody
  • Request a Review: Ask Kody to review your PR manually by adding a comment with the @kody start-review command at the root of your PR.

  • Validate Business Logic: Ask Kody to validate your code against business rules by adding a comment with the @kody -v business-logic command.

  • Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration
Review Options

The following review options are enabled or disabled:

Options Enabled
Bug ✅
Performance ✅
Security ✅
Business Logic ✅

Access your configuration settings here.

​

@coderabbitai

coderabbitai Bot commented Aug 16, 2026

Copy link
Copy Markdown

Important

Review available on request

  • 🔍 Trigger review

Reviews should be triggered manually for repositories with fewer than 10 stars. Select Trigger review above or comment @coderabbitai review to review the latest changes. For a full review, comment @coderabbitai full review.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: b48dc3b7-9c82-47e9-a66c-96f1db4739ed


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@laithanhhung
laithanhhung merged commit 295c51d into dev Aug 16, 2026
3 of 4 checks passed
@laithanhhung
laithanhhung deleted the update/layout branch August 16, 2026 18:48
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.

1 participant