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

release: 2.1.2 #989

Merged
merged 7 commits into from
Apr 2, 2024
Merged

release: 2.1.2 #989

merged 7 commits into from
Apr 2, 2024

Conversation

sakjung
Copy link
Collaborator

@sakjung sakjung commented Apr 2, 2024

구현 기능

sakjung and others added 6 commits April 15, 2023 16:01
* feat: 공간 예약 조건에 우선순위 개념 추가

우선순위를 가지는 조건이 모든 조건중에 우선적으로 적용된다

* feat: priority 중복 검증 로직 추가

* refactor: priority -> order 로 네이밍 변경

priority 는 높은 숫자가 우선인지 낮은숫자가 우선인지 모호할 수 있음

네이밍을 order로 하여 낮은 순서부터 우선순위를 가지는 점을 명백히 한다

* feat: settings flatten 메서드 생성

setting 검증의 용이함을 위해서 setting overwrite의 개념을 평탄화 하여 나타내주는 메서드

* feat: setting 예약 조건 요약 API 추가

* fix: test fail 수정

* test: settings flatten 관련 테스트 작성

* test: setting 요약 조회 API 인수 테스트 작성

* fix: SettingRequest DTO priorityOrder 필드명 변경

* fix: 클라측으로 반환시 priorityOrder 의 역순으로 정렬해서 반환

* refactor: 특정 요일에 대한 세팅만 추출해서 머지하도록 수정

예약 조건 검증 시, 요일에 대한 '실질적' 세팅 객체를 만들어서 검증

* feat: setting summary api - flat setting summary 요청 시, setting merge 로직 추가

* fix: setting summary api 에러 수정

Setting View Type 에 따른 로직 분기 ENUM으로 처리

* test: setting controller test case 고도화

* docs: flyway script 초기 priority order 값 수정

* feat: 예약 검증 관련 에러 메세지 수정
* feat: setting summary query 모든 예약 폼에 적용

서버측의 API 에서 받아온 메세지를 그대로 뿌려준다

* feat: 공간 예약 조건 우선순위 필드 추가해서 API 쏘도록 변경

마지막에 생선된 조건이 우선 적용됨 (= 낮은 priorityOrder)

* feat: 공간 수정 페이지에 안내문구 추가

* feat: 예약 조건 텍스트 보여주는 형식 수정

* feat: 공간 수정 화면에서 예약 조건 설정 관련 부가 기능 추가

현재 적용된 예약 조건 확인 기능 추가, 안내 메세지 추가
slack 서버의 네트워크 관련 구성, 설정등을 알 수 없음. 따라서, Reactor Netty Docs(https://projectreactor.io/docs/netty/1.0.21/reference/index.html\#faq.connection-closed) 의 내용을 토대로 추정하여 대응한다.
* feat: 홈페이지에서 마이페이지 진입점 추가

* feat: 주소에 따라 예약자 관련 페이지일 경우 예약자 마이페이지로 분기
@sakjung sakjung self-assigned this Apr 2, 2024
@sakjung sakjung merged commit a662d15 into main Apr 2, 2024
2 checks 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