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

Yozumina frontend #26

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

Squarecat-meow
Copy link

add: image preview before upload, upload thumbnail in chirp bubble
add dependencies: React-redux, @reduxjs/toolkit
edit: chirp component into bottom component

Copy link
Owner

@pbzweihander pbzweihander left a comment

Choose a reason for hiding this comment

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

redux가 꼭 필요할까요...? 꼭 state management를 외부 의존성 껴서 해야한다면 차라리 jotai라던가... 그리고 package.json이랑 yarn.lock 파일이 커밋이 안 되어있어요!

@pbzweihander
Copy link
Owner

react hook form을 떼버릴까 고민 중... (사유: 티르 님이 쓰지말라고함)

@Squarecat-meow
Copy link
Author

redux가 꼭 필요할까요...? 꼭 state management를 외부 의존성 껴서 해야한다면 차라리 jotai라던가... 그리고 package.json이랑 yarn.lock 파일이 커밋이 안 되어있어요!

아무래도 리덕스가 무겁긴 하죠...? 전역 상태관리를 페이지 단위로 바꾸고자 한다면 jotai나 recoil이 알맞을지도 모르겠네요!

@pbzweihander
Copy link
Owner

jotai나 recoil조차도 당장은 필요 없을 것 같다는게 제 생각입니다

@Squarecat-meow
Copy link
Author

Squarecat-meow commented Jun 23, 2024

Redux를 도입한 이유는 API를 통해 받아온 이미지가 여러장이 되면 URL들을 관리하기 편하게 하려고 도입을 결정했었습니다. 지금 당장은 store에 URL을 넣고, 이미지를 지우면 URL을 Reset하는 간단한 구현이지만 후에 이미지 갯수가 더 많아지면 코드상으로도 스케일링하기 용이할거라 생각했는데, 필요없다고 생각하신 이유가 궁금합니다.

@pbzweihander
Copy link
Owner

음 지금 이미지가 필요한게 new chirp 컴포넌트 하나밖에 없으니까 그냥 state로 관리해도 되는 것 아닌가요...?

@pbzweihander
Copy link
Owner

아 혹시 미스키마냥 다른 페이지에 갔다오거나 헀을 때도 첨부한 이미지가 남아있게 하기 위함인가요 그럼 인정

@pbzweihander
Copy link
Owner

제가 사실 redux jotai recoil 다 안 써봐서 웬만하면 안 붙이려고 한 건데 흠 이정도 규모의 프로젝트면 하나쯤 붙여야할 것 같기도 하네요

@pbzweihander
Copy link
Owner

함 jotai로 해보실래요?? 티르 님의 점지!!

@Squarecat-meow
Copy link
Author

jotai 정도면 useState문과 비슷하고 가볍고 타입스크립트 친화적이라고 하니 쓸만할 것 같네요!

@Squarecat-meow
Copy link
Author

todo: 이미지 업로드 취소기능, 올린 이미지를 노트 리스트에서도 볼 수 있게, 이미지 선택창 크기 정형화

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants