Skip to content

fix(orders): show orders without users#2906

Merged
Armanul46 merged 3 commits into
sovware:developmentfrom
syedgalib:fix/orders-show-userless
Jul 23, 2026
Merged

fix(orders): show orders without users#2906
Armanul46 merged 3 commits into
sovware:developmentfrom
syedgalib:fix/orders-show-userless

Conversation

@syedgalib

@syedgalib syedgalib commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Security fix
  • Improvement
  • New Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Text changes
  • Other... Please describe:

Description

Use a left join for admin order customer data so orders remain visible when the referenced user is missing. Show safe guest and missing-account fallbacks in the admin orders table and order detail sidebar without creating invalid user links.

How to reproduce the issue or how to test the changes

  1. Create or inspect orders for an existing user and for a guest, missing, or deleted user account.
  2. Confirm every order remains visible in the admin orders table, valid users retain their profile links, and missing users show the fallback labels.
  3. Open each order detail view and confirm customer information renders correctly without warnings or broken links.

Before after

https://prnt.sc/bKub3Fi4JN0M
orders no longer vanish from admin just because customer account data is missing.

Any linked issues

N/A

Checklist

@Armanul46 Armanul46 added this to the v8.8.8 milestone Jul 20, 2026
@RabbiIslamRony

Copy link
Copy Markdown
Member

@syedgalib, please remove the build, blocks, and POT files. Also, resolve any merge conflicts.

@RabbiIslamRony RabbiIslamRony left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks Good

@Armanul46
Armanul46 merged commit 4356dab into sovware:development Jul 23, 2026
1 check failed
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.

3 participants