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

[Refactor] 명세서랑 다른 점 수정 및 로직 변경 #13

Merged
merged 3 commits into from
Nov 27, 2023

Conversation

sss4920
Copy link
Contributor

@sss4920 sss4920 commented Nov 27, 2023

🚩 관련 이슈

📋 구현 기능 명세

  • main에서 가격 표시 수정
  • ticket에서 가격 표시 및 시간 정보 수정

📌 PR Point

  • 무슨 이유로 어떻게 코드를 변경했는지
    명세서와 다른 부분이 있어서 수정했습니다.

  • 개발하면서 어떤 점이 궁금했는지
    가격 관련된 부분을 따로 공통부분으로 모아야할지 고민이긴합니다.

📸 결과물 스크린샷

[ticket]

{
    "code": 200,
    "message": "티켓 선택 페이지 조회 성공",
    "data": {
        "airResultDto": {
            "dateDto": {
                "startDate": "2023-11-23 목",
                "endDate": "2023-11-30 목"
            },
            "airId": 10,
            "airName": "진에어",
            "startTime": {
                "startTime": "08:35",
                "endTime": "11:15",
                "during": "2시간 40분"
            },
            "endTime": {
                "startTime": "18:35",
                "endTime": "11:15",
                "during": "2시간 40분"
            }
        },
        "ticketListDto": [
            {
                "ticketId": 1,
                "companyName": "인터파크",
                "stars": 4,
                "card": "KEB 하나카드",
                "price": "467,100",
                "comment": 47819
            },
            {
                "ticketId": 2,
                "companyName": "트리플",
                "stars": 4,
                "card": "KEB 하나카드",
                "price": "467,900",
                "comment": 47819
            },
            {
                "ticketId": 3,
                "companyName": "마이리얼트립",
                "stars": 5,
                "card": "KEB 하나카드",
                "price": "467,900",
                "comment": 47819
            },
            {
                "ticketId": 4,
                "companyName": "노랑풍선",
                "stars": 3,
                "card": "KEB 하나카드",
                "price": "467,900",
                "comment": 47819
            },
            {
                "ticketId": 5,
                "companyName": "트립닷컴",
                "stars": 4,
                "card": "KEB 하나카드",
                "price": "467,900",
                "comment": 47819
            },
            {
                "ticketId": 6,
                "companyName": "참좋은여행",
                "stars": 3,
                "card": "KEB 하나카드",
                "price": "467,900",
                "comment": 47819
            },
            {
                "ticketId": 7,
                "companyName": "내일여행",
                "stars": 2,
                "card": "KEB 하나카드",
                "price": "467,900",
                "comment": 47819
            }        ]
    }
}

[main]

{
                "cityName": "일본",
                "country": "삿포로",
                "startDate": "2023-11-23 목",
                "endDate": "2023-11-30 목",
                "companyAir": "대한항공",
                "minPriceString": "467,100원 출발"
            }

🛠️ 테스트

  • 테스트

🚀 API Endpoint

  • main , ticket

@sss4920 sss4920 self-assigned this Nov 27, 2023
Copy link
Contributor

@mmihye mmihye left a comment

Choose a reason for hiding this comment

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

헉 그렇네; 큰일날뻔했네욤..

@sss4920 sss4920 changed the base branch from main to dev November 27, 2023 15:01
@sss4920 sss4920 changed the title Refactor/#12 [Refactor] 명세서랑 다른 점 수정 및 로직 변경 Nov 27, 2023
Copy link
Contributor

@mmihye mmihye left a comment

Choose a reason for hiding this comment

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

굿 👍🏻

@sss4920 sss4920 merged commit e47137c into dev Nov 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Refactor] 명세서랑 다른점 수정
2 participants