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

Feature/#196 페이지 title 서버인스턴스에 저장 #203

Merged
merged 5 commits into from
Nov 26, 2024
Merged

Feature/#196 페이지 title 서버인스턴스에 저장 #203

merged 5 commits into from
Nov 26, 2024

Conversation

hyonun321
Copy link
Collaborator

📝 변경 사항

Nov-26-2024.19-39-17.mp4
  • pageUpdateOperation 추가
  • title, icon 별로 update할 수 있도록 분리화
  • page title의 낙관적 업데이트 할 수 있도록 syncWithServer 매개변수 추가

🔍 변경 사항 설명

  • pageUpdateOperation을 추가했습니다.
  • 이에 따라 workspace에도 remotePageUpdate 메소드를 구현하혔습니다.

🙏 질문 사항

  • 아직 제목에 입력시 캐럿이 튀어버리는 버그가 있습니다..! 이부분은 추가로 잡을 예정입니다.
  • editor에 블럭이 생성이 안되면 제목으로 캐럿관리가 잘되는데, 블럭이 1개라도 생기는 순간 그쪽으로 다 이동해버립니다.. 마치 블랙홀처럼..

📷 스크린샷 (선택)

  • UI 변경이 있는 경우 스크린샷이나 GIF를 첨부합니다.

✅ 작성자 체크리스트

  • Self-review: 코드가 스스로 검토됨
  • Unit tests 추가 또는 수정
  • 로컬에서 모든 기능이 정상 작동함
  • 린터 및 포맷터로 코드 정리됨
  • 의존성 업데이트 확인
  • 문서 업데이트 또는 주석 추가 (필요 시)

@hyonun321
feat: pageUpdateOperation 구현
9da83e0 
- icon, title에 따라 업데이트 하도록 구현

#196
@hyonun321
chore: gateway내 함수 네이밍 수정
7360afd 
- 통일성 유지
@hyonun321
feat: 페이지 이름 변경시 pageUpdate하도록 수정
89415fa 
- 현재 페이지 이름 수정 후, 다른곳으로 focus가 빠지면 실행됨.
- 페이지 이름까지 실시간 수정이 이루어질 필요는 없다 판단하여 onBlur를 사용.

#196
@hyonun321
refactor: page Title 업데이트를 낙관적 업데이트로 변경
10dcef1 
- 유저에게는 바로 반영
- 실제 서버 저장은 title에서 focus 가 날아갈때 실행
- 현재 제목에 입력할시 caret이 editor로 날아가는 문제 있음

#196
@hyonun321 hyonun321 added FE 프론트엔드 작업 BE 백엔드 작업 Feat 새로운 기능 추가나 기존 기능 확장 작업 labels Nov 26, 2024
@hyonun321
Merge branch 'dev' of https://github.com/boostcampwm-2024/web33-Nocta …
5cee5bc 
…into Feature/#196_페이지_title_서버인스턴스에_저장

- 병합하며 iconType interface 일치화
- build 에러 수정

#196
Ludovico7
Ludovico7 approved these changes Nov 26, 2024
View reviewed changes
Copy link
Collaborator

@Ludovico7 Ludovico7 left a comment

Choose a reason for hiding this comment

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

확인했습니다 고생하셨습니다!

minjungw00
minjungw00 approved these changes Nov 26, 2024
View reviewed changes
Copy link
Collaborator

@minjungw00 minjungw00 left a comment

Choose a reason for hiding this comment

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

수고많으셨습니다!

@github-actions github-actions bot merged commit 65bc6b6 into dev Nov 26, 2024
5 checks passed
@hyonun321 hyonun321 self-assigned this Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Ludovico7 Ludovico7 Ludovico7 approved these changes

@minjungw00 minjungw00 minjungw00 approved these changes

@pipisebastian pipisebastian Awaiting requested review from pipisebastian

Assignees

@hyonun321 hyonun321

Labels
BE 백엔드 작업 FE 프론트엔드 작업 Feat 새로운 기능 추가나 기존 기능 확장 작업
Projects
None yet
Milestone
페이지를 생성하고 관리할 수 있다.
Development

Successfully merging this pull request may close these issues.
3 participants
@hyonun321 @minjungw00 @Ludovico7