Skip to content

fix(products): cache product list as array to avoid model serialization issues - #359

Merged
trtajim merged 1 commit into
mainfrom
fix/products-cache-array
Sep 21, 2026
Merged

trtajim merged 1 commit into
mainfrom
fix/products-cache-array

Conversation

@trtajim

@trtajim trtajim commented Sep 21, 2026

Copy link
Copy Markdown
Member

Summary

  • Cleared stale serialized product cache.
  • Updated ProductController::index to cache query results as arrays (->get()->toArray()) rather than Eloquent model collections to avoid PHP serialization mismatches and stale schema state across requests.

Checks

  • Ran npm run format && composer lint && npm run lint
  • Ran php artisan test (all 262 tests passed)

@coderabbitai

coderabbitai Bot commented Sep 21, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 36 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository: hscstack/platform/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 39756050-3111-4129-91d9-146033b1cfbb

📥 Commits

Reviewing files that changed from the base of the PR and between 2e0de8e and e955b2b.

📒 Files selected for processing (1)
  • app/Http/Controllers/ProductController.php

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.

@trtajim
trtajim merged commit 2eeecf5 into main Sep 21, 2026
6 checks passed
@trtajim
trtajim deleted the fix/products-cache-array branch September 21, 2026 16:33
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