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.

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

refactor: 이력서 작성 페이지 msw 작성 #289

Merged
merged 10 commits into from
May 1, 2024

Conversation

leeminhee119
Copy link
Member

💻 스크린샷 (선택사항)

📃 PR 설명

  • 6 카테고리 작성 시 낙관적 업데이트가 작동합니다.

📃 참고사항

  • 참고링크, 기본정보 작성은 낙관적 업데이트가 작동하지 않습니다. 따라서 저장을 눌러도 입력한 값이 보이지 않습니다.
  • 기존에 페이지 Loader 작동을 막기 위해 router에서 주석 처리하고 Fragment로 바꿔놨는데, 아무것도 뜨지 않게 돼버려서 다시 원상복구하고 로더 내부에서 주석 처리해두었습니다.

🔎 PR포인트 및 궁금한 점

  • 참고링크 mock 데이터 작성해서 네트워크 탭에서 데이터 잘 오는 것까지 확인했는데, 왜인지 렌더링할 때는 빈 배열로 읽어버리네요.

@leeminhee119 leeminhee119 added the 🛠 Refactor 리팩토링한 경우(리팩토링 과정에서 파일 삭제, 이동 포함) label Apr 26, 2024
@leeminhee119 leeminhee119 requested a review from khakhiD April 26, 2024 11:25
@leeminhee119 leeminhee119 self-assigned this Apr 26, 2024
Copy link
Contributor

@khakhiD khakhiD left a comment

Choose a reason for hiding this comment

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

고생하셨습니다. 어프루브가 늦어졌어요.

@leeminhee119 leeminhee119 merged commit fc0b7da into main May 1, 2024
1 check passed
@leeminhee119 leeminhee119 deleted the refactor/resume-mock branch May 1, 2024 08:57
@leeminhee119 leeminhee119 linked an issue May 1, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠 Refactor 리팩토링한 경우(리팩토링 과정에서 파일 삭제, 이동 포함)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

refactor: 이력서 mock 데이터 작성
2 participants