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

feat: 공간 조건 적용 사용성 개선 #963

Merged
merged 16 commits into from
Apr 15, 2023

Conversation

sakjung
Copy link
Collaborator

@sakjung sakjung commented Apr 11, 2023

구현 기능

  • 공간 예약 조건에 우선순위 개념 부여
    • 나중에 만들어진 조건일수록 우선순위를 가진다
    • 우선순위가 높은 조건이 낮은 조건을 덮어 쓴다
    • priorityOrder가 낮을수록 더 높은 우선순위
  • 공간 예약 조건 overwrite 개념을 단순화하기 위한 stack, flat 개념 도입
    • stack: 공간 관리자가 설정한 그대로의 값 (=overwrite 를 아직 고려하지 않은 공간 예약 조건)
      • 공간 관리자가 설정한 그대로의 값이기 때문에 stack 상태의 예약조건들은 서로 겹칠 수 있다
    • flat: 실제 예약 시 적용될 조건 (=overwrite 를 고려한 공간 예약 조건)
      • 서로 겹치는 상태가 없는 조건들. flat한 상태의 공간 예약 조건들은 서로 상호배타적, 독립적이다
    • 실제 예약과 관련해서는 flat 상태의 에약 조건만 고려하면 된다 (flat 조건이 젤 중요)
    • stack 은 순수하게 관리자용
  • 공간 예약 조건 API 추가

논의하고 싶은 내용

  • 바쁘시니 리뷰는 괜찮구, 구현 기능에서 언급도니 개념 참고 정도 부탁드립니다

공유하고 싶은 내용

Close #960
Close #961

sakjung added 14 commits April 2, 2023 01:01
우선순위를 가지는 조건이 모든 조건중에 우선적으로 적용된다
priority 는 높은 숫자가 우선인지 낮은숫자가 우선인지 모호할 수 있음

네이밍을 order로 하여 낮은 순서부터 우선순위를 가지는 점을 명백히 한다
setting 검증의 용이함을 위해서 setting overwrite의 개념을 평탄화 하여 나타내주는 메서드
예약 조건 검증 시, 요일에 대한 '실질적' 세팅 객체를 만들어서 검증
Setting View Type 에 따른 로직 분기 ENUM으로 처리
@sakjung sakjung merged commit 154cb8a into dev Apr 15, 2023
@sakjung sakjung deleted the feat/960-space-setting-overwrite branch April 15, 2023 07:01
sakjung added a commit that referenced this pull request Aug 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🕋 backend 🌟 feature New feature or request
Projects
None yet
1 participant