Skip to content

Fix banner service by populating Order.confirmedAt timestamp#130

Closed
Copilot wants to merge 2 commits intodevelopfrom
copilot/fix-home-banner
Closed

Fix banner service by populating Order.confirmedAt timestamp#130
Copilot wants to merge 2 commits intodevelopfrom
copilot/fix-home-banner

Conversation

Copy link
Copy Markdown

Copilot AI commented Feb 13, 2026

Problem

The banner service queries orders by confirmedAt to determine review prompt eligibility, but this field was never populated. All banner queries returned empty results.

Changes

Added confirm() method to Order entity:

public void confirm() {
    this.orderStatus = OrderStatus.CONFIRMED;
    this.confirmedAt = LocalDateTime.now();
}

Follows existing cancel() pattern. Enables banner service to:

  • Show initial review prompts after order confirmation
  • Show monthly review prompts 5+ days post-confirmation

Modified: src/main/java/com/ongil/backend/domain/order/entity/Order.java

Original prompt

This section details on the original issue you should resolve

<issue_title>[fix] 홈화면 배너</issue_title>
<issue_description>## 📌 Description

[fix] 홈화면 배너

✅ To-do

  • 진행 예정
  • 진행 완료

📒 기타</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: neibler <87866961+neibler@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix home screen banner issue Fix banner service by populating Order.confirmedAt timestamp Feb 13, 2026
Copilot AI requested a review from neibler February 13, 2026 08:28
@neibler neibler closed this Feb 13, 2026
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.

[fix] 홈화면 배너

2 participants