[#186] PWA 적용 #189
Merged
[#186] PWA 적용 #189
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
kimyou1102
reviewed
Jul 13, 2024
src/app/chat/page.tsx
Outdated
| @@ -24,7 +24,7 @@ const ChatList = () => { | |||
|
|
|||
| useEffect(() => { | |||
| if (!getItem(localStorage, 'token')) router.replace('/signin'); | |||
| }, [router]); | |||
| }, [router, chatRooms]); | |||
There was a problem hiding this comment.
페이지 리렌더링 시 토큰 검사를 하는 것이라 생각했는데 chatRooms를 의존성 배열에 추가시킨 이유가 궁금합니다!
There was a problem hiding this comment.
저게 어쩌다 저기 들어갔는데 여러본 확인한 결과 잘못들어간것이었습니다. 죄송합니다
| @@ -11,5 +11,5 @@ export async function POST(req: NextRequest) { | |||
| console.log('FCM Error: ', e); | |||
| } | |||
|
|
|||
| return; | |||
| return Response.json({}); | |||
There was a problem hiding this comment.
잘은 모르지만 클라이언트에서 sendPostFCM함수로 api통신 시 이 파일의 코드를 실행하고, 이 통신의 결과를 onSuccess와 onError에서 처리를 하는 거라고 이해했습니다.
제가 이해하는 게 맞다면 이렇게 하면 에러가 발생하더라도 빈구문이 넘어와서 명확하게 에러를 인지하기 어려울 거 같아요.
더 명확하게 에러가 발생했다는 것을 알리기 위해 catch블록 안에 에러 응답을 반환하는 것도 좋을 거 같아요!
} catch (e) {
console.log('FCM Error: ', e);
return new Response(JSON.stringify({ error: 'FCM notification failed', details: e.message }), {
status: 500,
headers: { 'Content-Type': 'application/json' },
});
}
YunSukHyun
force-pushed
the
feat/#186/apply-pwa
branch
from
47946db to
8401d79
Compare
kimyou1102
approved these changes
Aug 12, 2024
kimyou1102
pushed a commit
that referenced
this pull request
Aug 19, 2024
kimyou1102
added a commit
that referenced
this pull request
Sep 6, 2024
* refactor: 주소변환 api 변경 - 주소 검색과 같은 양식의 결과값을 위해 useGetAddressByCoordinates훅의 geocoder 메서드를 coord2Address로 변경 - coord2RegionCode 메서드를 이용한 주소반환 훅을 새로 생성 - location.city값으로 모각코 리스트 썸네일용 행정동 주소를 전달 * feat: 건물이 아닌 곳에서 모각코 생성 비활성화 - useRef를 사용해 최초 렌더시에만 건물인 곳에도 마커가 찍히게 구현 * modify: 목록보기 및 모각코 생성버튼을 포함한 HomeMapFooter컴포넌트 생성 * fix: 바텀시트 오픈시 '목록보기'버튼과 모각코 생성버튼이 클릭되지 않는 문제 해결 * fix: 위치정보 미허용시 지도에 마커와 주소가 뜨지 않던 문제 해결 - 위치정보 미허용시 기본 위치에 마커와 주소 표시되도록 수정 * modify: 서치바 외부 클릭시 주소목록 컴포넌트가 사라지도록 수정 * modify: 도로명 주소 기준으로 넘겨주도록 주소변환 커스텀훅 응답값 변경 - 커스텀훅 응답값 변경 - LocationInfo에 도로명 주소를 담을 hCity 추가 * fix: 롱프레스시 제대로 반영 안되던 문제 해결 - 인포윈도우를 이동시킨 위치에서 반영 안되던 문제도 해결 * feat: 모각코 생성페이지 새로고침 및 페이지 이탈 막기 * feat: 건물이 아닌곳에 롱프레스 시 모각코 생성 비활성화 * fix: 모달에서 토스트 클릭시 모달 닫히는 오류 해결 * fix: 모각코 수정시 라디오그룹 디폴트값이 반영되지 않던 문제 해결 * feat: KakaoMapService 초기화 시 clusterer 추가 - KakaoMapService가 context를 통해 map과 clusterer를 관리하도록 개선 * [#186] PWA 적용 (#189) * feat: pwa in chat * fix: pwa applyMGC * fix: fcm * fix: etc * fix: aws url * fix: 테스트 배포오류 * [#191] google analytics 연동 (#195) * chore: @next/third-parties, @types/gtag.js라이브러리 설치 * chore: @types/gtag.js 타입 추가 * feat: RootLayout에 GoogleAnalyticsTracker컴포넌트 추가 - GoogleAnalytics 컴포넌트를 사용하여 Google Analytics 설정 - 경로(pathname) 변경 시 페이지뷰 이벤트 호출 * feat: '참여하기' 버튼에 Google Analytics 이벤트 추적 추가 - '참여하기' 버튼 클릭 시 event 함수 호출 * 요청된 오류 수정 (#197) * remove: delete unused api * fix: signup Validation * fix: 10개이상의 채팅 불러오기 * fix: mogakkodetail * fix: aws url * fix: NickName edit * fix: localstorage error * feat: 번개모각코일 경우 수정 막기 * modify: 통합 검색어 입력시 최소길이 2로 제한 - placeholder에 두글자 이상이라는 문구 추가 * modify: 모각코 리스트의 주소 관련 data 속성 변경 - address는 도로명주소를 나타내며, city와 hCity에는 동명을 나타냄 - 주소를 나타내는 속성을 address에서 city로 변경 * feat: 문의 입력 시 최대 길이 255자로 제한 * refactor: searchValue 업데이트 방식 변경 - 주소를 지역 코드로 변환한 후, 원본 주소의 길이만큼 잘라서 searchValue의 address 값을 업데이트 * modify: 통합검색 placeholder 수정 * modify: 콜백함수 이름 변경 * modify: searchValue의 address값 법정동이 아닌 행정동으로 변경 * modify: 더미데이터와 mock 데이터에 속성 추가 - location 객체에 속성 추가로 더미데이터에 city, hCity 속성 추가 - mock 데이터에 hCity 속성 추가 * fix: 모바일 줌인/줌아웃 시 인포윈도우 생성되는 문제 해결 - 멀티 터치 발생 시 타이머를 초기화하도록 `handleTouchStart`에 터치 포인트 검증 추가 * feat: 모각코 생성 및 수정 페이지에서 뒤로가기 방지 및 모달 표시 - 페이지 이동 기록을 세션 스토리지에 저장하여, 뒤로 가기 시 이전 페이지로 이동 - 변경 사항이 없을 경우, 세션 스토리지의 기록을 기반으로 자동으로 이전 페이지로 이동 --------- Co-authored-by: SukHyun Yun <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📝 주요 작업 내용
📷 스크린샷 (선택)
💬 고민 중인 부분 및 참고사항 (선택)
close #186