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

feat: 일기 작성 기능을 구현합니다. #70

Merged
merged 9 commits into from
Aug 2, 2024

Conversation

wokbjso
Copy link
Member

@wokbjso wokbjso commented Jul 31, 2024

🤔 어떤 문제가 발생했나요?

🎉 어떻게 해결했나요?

  • 일기 작성 section 관련 기능들을 구현하였습니다.

📷 이미지 첨부 (Option)

default.mp4

⚠️ 유의할 점! (Option)

  • 현재는 우선 기능 구현이 우선이라 일기 내용이 바뀔 때마다 form 정보를 업데이트 해주고 있는데요, 외부에서 하나의 상태를 추가한 후 일기 내용이 바뀔때마다 해당 상태가 변화 / 실제 form 정보의 변화에는 debounce 처리 -> 이런 방식으로도 고민중에 있습니다.
  • form 정보의 업데이트가 잦을 시 성능 이슈를 좀 알아보고 위처럼 변경 여부를 정해야 할 것 같습니다~!!

@king2dwellsang king2dwellsang added Review Plz🙏 # Review is not yet complete feat # Add feature labels Jul 31, 2024
Copy link
Member

@Jungjjeong Jungjjeong left a comment

Choose a reason for hiding this comment

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

👍


const handleTextAreaChange = (text: string) => {
setValue('diary', text);
};
Copy link
Member

Choose a reason for hiding this comment

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

상태 변경에 debounce 넣는 부분 동의합니다 👍

setValue에 대한 상태값이 너무 많아서.. iOS 키보드로 입력이 잘 될지 모르겠어요!
요 부분 모웹에서의 테스트가 필요할 것 같습니다.

Copy link
Member

Choose a reason for hiding this comment

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

추가로 watch는 상태가 변경되는 부분을 계속 감시하고 있기 때문에 성능에 안좋을 수 있습니다.
그 수가 많아질수록 성능 향상을 위해 해줘야 하는 작업이 늘어날 것 같아요.

추후 상태 객체 형태로 변경 + register을 활용하여 watch 메소드를 활용하지 않도록 구현해야 할 것 같습니다.

Copy link
Member Author

Choose a reason for hiding this comment

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

추후 input 태그에 register 부여를 위해 text-field 컴포넌트 수정 PR 올리도록 하겠습니다!!

@king2dwellsang king2dwellsang added Approved 🆗 # Review is completed and removed Review Plz🙏 # Review is not yet complete labels Aug 2, 2024
@wokbjso wokbjso merged commit a32c541 into feat/record-photo Aug 2, 2024
2 checks passed
@wokbjso wokbjso deleted the feat/record-diary branch August 2, 2024 03:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved 🆗 # Review is completed feat # Add feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants