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

Backend/feature/25 amadeus api #66

Merged
merged 23 commits into from
May 12, 2024

Conversation

Dayon-Hong
Copy link
Collaborator

@Dayon-Hong Dayon-Hong commented May 12, 2024

#️⃣연관된 이슈

#45

📋 PR 개요

항공권 관련 API

✅ PR 유형

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

✏️ 변경 사항

    1. 항공 정보 조회 API
    1. 스카이스캐너 url 반환 API
    1. 항공권 좋아요 API
    1. 항공권 좋아요 목록 조회 API
    1. 항공권 좋아요 삭제 API

1. == 연산자를 통한 null 처리 -> Optional 처리
2.  flightService 내에 있는 findAirportCode의 접근 제어자 -> private으로 변경
1. == 연산자를 통한 null 처리 -> Optional 처리
2.  flightService 내에 있는 findAirportCode의 접근 제어자 -> private으로 변경
1. == 연산자를 통한 null 처리 -> Optional 처리
2.  flightService 내에 있는 findAirportCode의 접근 제어자 -> private으로 변경
@Dayon-Hong Dayon-Hong self-assigned this May 12, 2024
@Dayon-Hong Dayon-Hong merged commit 50e1d5f into dev_backend May 12, 2024
1 check passed
@Dayon-Hong Dayon-Hong deleted the backend/feature/25-amadeus-api branch May 12, 2024 14:32
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.

1 participant