Skip to content

commit: T225 approval row documentType parity and dashboard quick acc… - #64

Merged
ThienlocTran merged 1 commit into
devfrom
feature/T225-inventory-dashboard-layout
Aug 4, 2026
Merged

ThienlocTran merged 1 commit into
devfrom
feature/T225-inventory-dashboard-layout

Conversation

@25ThienNgoc

@25ThienNgoc 25ThienNgoc commented Aug 1, 2026 •

Copy link
Copy Markdown
Collaborator

check giúp tui

Summary by CodeRabbit

  • Cải tiến
    • Cập nhật danh sách phiếu duyệt để hiển thị đúng loại phiếu theo từng dòng.
    • Điều chỉnh tiêu đề và nhãn trong cửa sổ chi tiết theo loại phiếu đang xem, gồm “Khách hàng” cho phiếu xuất và “Nhà cung cấp” cho phiếu nhập.
    • Danh sách phiếu chỉ hiển thị sau khi tải dữ liệu hoàn tất.
    • Khu vực “Truy cập nhanh” hiển thị thông báo phù hợp khi không có mục nào được phép truy cập.

@coderabbitai

coderabbitai Bot commented Aug 1, 2026 •

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 675838d1-5e11-4d8f-9f26-fe2801919e09

📥 Commits

Reviewing files that changed from the base of the PR and between 7b51677 and b986442.

📒 Files selected for processing (2)
  • src/views/ApprovalsView.vue
  • src/views/DashboardView.vue

📝 Walkthrough

Walkthrough

Cập nhật giao diện duyệt phiếu để dùng đúng loại phiếu theo từng bản ghi và phiếu đang xem. Dashboard hiển thị trạng thái rỗng khi không có mục Quick Access được phép truy cập.

Changes

Cập nhật hiển thị giao diện

Layer / File(s) Summary
Hiển thị loại phiếu và chi tiết duyệt
src/views/ApprovalsView.vue
ApprovalsView dùng loại phiếu theo từng dòng và phiếu đang xem. Nhãn đối tác đổi giữa “Khách hàng” và “Nhà cung cấp”. Bảng chỉ hiển thị sau khi tải xong.
Trạng thái rỗng Quick Access
src/views/DashboardView.vue
Khi không có liên kết được phép truy cập, giao diện hiển thị thẻ thông báo quyền truy cập.

Estimated code review effort: 2 (Simple) | ~15 minutes

Possibly related PRs

Suggested reviewers: thienloctran, tonfa137206

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Tiêu đề nêu đúng hai thay đổi chính: đồng bộ documentType trong dòng phê duyệt và trạng thái trống cho Truy cập nhanh.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/T225-inventory-dashboard-layout

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.

@ThienlocTran ThienlocTran left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Oke

@ThienlocTran
ThienlocTran merged commit c2ccde8 into dev Aug 4, 2026
1 check passed
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.

2 participants