[Mission002] TODOS 시스템에 비동기/온오프라인을 고려하여 업데이트하기

[YongHoonJJo]

7 fetch를 이용해 데이터 CRUD 하기

1. 주어진 API 를 통해 CRUD 업데이트
2. async-await 을 통한 처리 및 그에 따른 try-catch 적용
3. 초기 앱 실행시 Get 을 통해 데이터를 가져오기 때문에, LocalStorage 는 적용하지 않았습니다.

mission01 작업한 곳에서 브랜치를 생성했는데, 기존 미션이 머지가 되지 않아, Mission02 로 된 부분을 보셔야 할 것 같습니다. ㅜㅜ

[YongHoonJJo]

@s280493

1. BaseURL 분리
2. Constants.js에서 만들어둔 상수 활용
3. data-action=edit 제거. id 로 구분하기
4. ${isCompleted && 'class="completed"'} 삼항연산자로 변경 처리
5. { completed ? "checked" : "" } 삼항연산자로 변경 처리

제가 브랜치 생성하는 과정을 착각해서, 미션2를 이렇게 진행한 것 같습니다. 다음부터는 브랜치 잘 생성해서 진행하겠습니다 .ㅜㅜ 리뷰 감사합니다!!

[YongHoonJJo]

@EastjunDev

1. eventHandler 분리
2. Counter 대신 Count 로 네이밍 변경하였습니다.
3. removeState 대신 removeItem 으로 네이밍 변경
4. this.index = 0 사용하지 않는 변수 제거
5. 컴포넌트 네이밍 관련, $표시로 인한 *Element 제거

mission1 머지 되면, mission2 에도 mission1 에서 변경된 사항 적용한다음 머지시키면 문제 없을 것 같습니다. 제가 브랜치를 잘못생성해서 번거롭게 되었네요 ㅜㅜ

res 변수를 재사용하지 않는다면, fetch하는 부분에서
fetch(uri, config).then((response) => response.json())
위와 같이 변경하면 보다 좋을 것 같아요~!

라고 해주신 부분은 이해가 잘 안되서, 답글 남겼는데, 아직 답변이 없으셔서 이 내용에 대해서는 적용하지 못하였습니다.ㅜㅜ

===가 아닌 ==로 하신 이유에 대한 문의도, 답글 남겼는데, 아직 답변이 없으셔서 적용하지 못했습니다!!

리뷰 감사합니다!!!!