Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: "group by" error #407

Merged
merged 12 commits into from
Jan 20, 2025
Merged

fix: "group by" error #407

merged 12 commits into from
Jan 20, 2025

Conversation

fox-john
Copy link
Contributor

@fox-john fox-john commented Jan 17, 2025

  • Fix of the count query for the incremental processing of certain shop content ;
  • Segmentation of shop content for Orders, OrderCarriers, OrderCartRules, OrderDetails, and OrderStatusHistories. Each now has its own hook. Consequently, the "limit" parameter has been reactivated for these shop contents in incremental sync ;
  • Minor fix for the integration of accounts (an issue that only affected E2E tests) ;
  • PHP doc fixes.

@fox-john fox-john self-assigned this Jan 17, 2025
@fox-john fox-john added the 🥕 Radis to review Ready to reviewed by 1 peer label Jan 17, 2025
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
3.3% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@fox-john fox-john merged commit 62cd09b into main Jan 20, 2025
21 of 23 checks passed
@fox-john fox-john deleted the fix/group-by-error branch January 20, 2025 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🥕 Radis to review Ready to reviewed by 1 peer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants