codestates / medisharp-client

🗓 촬영 한 번으로 약 정보를 얻고, 해당 약의 복용 일정도 관리할 수 있는 모바일 서비스 '약올림'.💊 건강도, 편리함도 한 단계 올려보세요.
2 stars 0 forks source link

[Client]Alpha/error fix 1223 #113

Closed hyunju-song closed 3 years ago

hyunju-song commented 3 years ago
  1. 회원가입/ 아이디 찾기에서 버그 발생시키는 코드 다시 수정했습니다
  2. 에러 alert 관련하여 없어진 부분 다시 추가했습니다.
  3. 알람 수정/ 등록 동기적 분기 좀더 명확하게 하였고, 푸쉬 코드 수정했습니다.

제 생각에는 작업하다가 merge 하지 않고 pr 하여 코드가 없어진게 아니라, 1번이 pr하고 -> 2번이 작업한거 pr하고 먼저 merge하고 -> 1번이 merge 해서 더 과거작업인 1번으로 덮여서 기존 작업 코드가 사라진듯 합니다. 따라서 pr 했더라도 아직 머지하지 않은 상태에서 이슈가 새로 merge 되면 pull-> merge 하는게 좀더 정확할듯합니다!