search

baesy0 / kalena

Kalena is calendar web application based on opensource.
BSD 3-Clause "New" or "Revised" License
6 stars 4 forks source link

이름이 일치하는 레이어가 있는지 체크 #232

Closed baesy0 closed 4 years ago

baesy0 commented 4 years ago

레이어 이름이 존재하지 않으면 스케쥴이 추가되지 않도록 에러처리 함. 현재는 터미널에서 스케쥴을 추가할 때, 존재하지 않는 레이어이름을 입력해도 스케쥴이 추가됨.

baesy0 commented 4 years ago

close: #213

baesy0 commented 4 years ago

웹페이지에서 스케쥴을 추가할 땐 레이어의 목록이 드롭다운으로 생성되기 때문에 상관없지만, 커맨드라인으로 스케쥴을 추가할 땐 존재하지 않는 레이어이름을 넣을 수 있더라구요. 그 경우를 걸러내기 위해 추가했어요

baesy0 commented 4 years ago

로그 삭제했어요!

khw7096 commented 4 years ago

오프라인 리뷰를 해봤습니다.

  1. 레이어가 존재하는지만 체크하는 함수를 만듭니다. AddSchedule 함수 내부에서 레이어가 존재하는지 체크하는 건 이상하다고 생각했어요.
  2. 레이어를 체크하는 함수는 restAPI에서 필요할 때 호출해서 사용하면 될것 같아요.
baesy0 commented 4 years ago

AddSchedule은 스케쥴을 추가하는 기능만 수행하도록 남기고 레이어가 존재하는지 체크하는 부분은 restAPI에서 스케쥴 추가하기 전에 체크하도록 옮겼어요