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.

Sign up for GitHub

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

Jump to bottom

my room 메모 CRUD 기능 #117

Merged
merged 22 commits into from
Aug 16, 2022
Merged

my room 메모 CRUD 기능 #117

merged 22 commits into from
Aug 16, 2022

Conversation

Bhanjo
Copy link
Contributor

@Bhanjo Bhanjo commented Jul 26, 2022 •
edited
Loading

캡처1
캡처2
캡처3

[Todo]

  • 메모장 5장 미리 보기
  • 메모장 전체 보기
  • 메모장 삭제
  • 메모장 추가
  • 메모장 5장 미만 뷰 로직 개선
  • 유저에 따른 메모 수정, 삭제 기능 추가

[ISSUE]

  • 메모장 미리 보기, 전체 보기 api가 완전해지면 테스트 필요
  • 메모장, 룸 삭제 기능 api 에러로 불가
  • 유저 타입별 접근 권한 부여 어떻게 할 것인지 논의 필요

@Bhanjo
Merge remote-tracking branch 'origin/develop' into feature/hanjo/view…
0b5b477 
…MyRoomData
@Bhanjo
style: 폴더명 변경
31060a6
@Bhanjo
feat: room 데이터 패치 기능 구현
18d2107
@Bhanjo
feat: 로그인 검증 구현
e943e4b
@Bhanjo
feat: 메모 등록 기능 구현
27e560c
@Bhanjo
feat: 메모장 전체 보기 기능 구현
b142b8e
@Bhanjo
Merge remote-tracking branch 'origin/develop' into feature/hanjo/view…
0db626c 
…MyRoomData
@Bhanjo
refactor: api 분리, 리액트 쿼리를 커스텀 훅으로 리팩토링
3a8b081
@Bhanjo
refactor: 메모 등록 로직 리액트 쿼리로 리팩터링
8a4cc8d
@Bhanjo
fix: 간혈적으로 리패치 안 되는 현상 해결, 메모장 최신순 정렬
042c1ad
@Bhanjo
style: 변수명 직관적으로 변경, 토스트 메시지 등장 시기 변경
8efa798
@Bhanjo
fix: 일부 메모 내용이 안보이는 현상 해결
6b735ed
@Bhanjo
setting: await expression 규칙 추가 (겁나 중요)
84a680c
@Bhanjo
fix: 데이터 리패칭 오류 해결
ef0ca72
@Bhanjo
feat: 메모장 수정 기능 구현
732ce46
@Bhanjo Bhanjo requested a review from a team August 14, 2022 05:21
syeon2565
syeon2565 reviewed Aug 14, 2022
View reviewed changes
Copy link
Member

@syeon2565 syeon2565 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

굿굿 수고하셨습니다!

방명록 5개 이하일때는 방명록이 추가되면 보드컴포넌트가 렌더링되는것도 고려해봐도 좋을 것 같습니다!

src/pages/rooms/[id].tsx Show resolved Hide resolved
src/components/Rooms/MemoModal.tsx Outdated Show resolved Hide resolved
src/pages/rooms/[id].tsx Outdated Show resolved Hide resolved
@Bhanjo
Copy link
Contributor Author

Bhanjo commented Aug 14, 2022

굿굿 수고하셨습니다!
방명록 5개 이하일때는 방명록이 추가되면 보드컴포넌트가 렌더링되는것도 고려해봐도 좋을 것 같습니다!

이건 캐시 배우고 적용해보겠습니다!

@Bhanjo Bhanjo requested a review from a team August 14, 2022 15:18
@Bhanjo Bhanjo merged commit dc9aa43 into develop Aug 16, 2022
@Bhanjo Bhanjo deleted the feature/hanjo/viewMyRoomData branch August 16, 2022 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@syeon2565 syeon2565 syeon2565 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Bhanjo @syeon2565