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

fix : 유저 피드백에 의한 기능 추가 #199

Conversation

Banal972
Copy link
Contributor

#️⃣연관된 이슈

#192

📝작업 내용

  • 로그아웃을 해도 '찜한 모임' 탭 옆에 숫자가 떠 있습니다.

userToken을 이용해서 로그인을 하지 않는경우 뜨지 않게 수정 하겠습니다.

  • 로그인을 하지 않았는데 찜 기능이 작동하는 것도 수정이 되어야할 것 같아요

유저 토큰을 가지고 있으면 막는 형식으로 수정하겠습니다.

  • 모임 개설시 시간이 제대로 반영이 안됩니다.

확인해보고 수정

  • alert가 쿼리파라미터로 들어가는데 이게 url 히스토리에 남아서 모임찾기에서 join을 누르고 난 뒤에 뒤로가기를 두번

미처 join에서의 버그는 발견하지 못한것 같습니다. 수정 하겠습니다.

  • 찜한 모임에서 JOIn now누르면 모임찾기로 넘어감

찜한 모임에서 join now누르면 모임찾기로 넘어감
라우터가 지정되어있어
pathname으로 해당 라우터에서 작동하기 수정

  • 모임에 대한 구분을 생성하는 기능이나 지인으로 하여금 모임에 참여할 수 있도록 모임 참여에 대한 qr이나 링크를 공유할 수 있는 기능이 있었으면 좋겠네요.

개설자는 링크공유를 할 수 있는 버튼이 존재하는데
따로 개설자가 아니면 존재 하지 않아서
꼭 개설자가 아니여도 버튼이 나오게 수정하겠습니다

  • 검색 필터 적용 후 "해제" 버튼

추가하기

  • 리뷰, 모임찾기 등 데이터가 없을경우

'등록된 모임이 없어요' 추가 예정

  • 모임 개설 시 이미지 미리보기도 가능했으면 좋겠어요!

추가해보겠습니다

추가가 불가능한것

  • 개설된 모임의 주인이 누구인지가 노출이 되면 더 좋을 것 같아요.

API에 해당 유저를 가져오는 response가 없어서 불가능

  • 모든 리뷰에서 내용이 없어도 스켈레톤 UI가뜸

뜨지 않는것으로 보아 인터넷 문제거나 로딩이 오래걸렸던것 같음

  • 내 모임에서 모임 취소를 누르면 모임 상세로 들어가지는데 거기서 취소를 누르고 뒤로가기를 누르면 내 모임에 취소된 게 바로 반영이 되지 않습니다. (새로고침을 해야 반영이 되는 듯 합니다)

정헌님이 작업하셔서 이월 했습니다.

스크린샷 (선택)

리스트가 없을경우
image

필터 초기화
image

로그인을 하지 않았을경우 찜을 못하게 막았습니다.
찜한 모임에서 join now누르면 모임찾기로 넘어감

라우터가 지정되어있어

pathname으로 해당 라우터에서 작동하기 수정
데이터가 존재하지 않을경우 문구를 추가하였습니다.
어떤 상황에서도 공유하기 버튼은 나오도록 수정
모임 찾기 시간 반영 제대로 안되는 버그 수정
@Banal972 Banal972 self-assigned this Aug 22, 2024
@Banal972 Banal972 added feat 새로운 기능 추가 fix 오류를 수정함 labels Aug 22, 2024
Copy link
Contributor

@changmin6362 changmin6362 left a comment

Choose a reason for hiding this comment

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

아프신데 고생하셨습니다.

Copy link
Contributor

@JeongHeonK JeongHeonK left a comment

Choose a reason for hiding this comment

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

아예 페이지 새로 만드셨네요 고생하셨습니다!

@@ -106,6 +109,12 @@ const CreateMeetingForm = ({ changeState }: { changeState: () => void }) => {
if (!file) {
setFileName("")
} else {
const fileReader = new FileReader()
Copy link
Contributor

Choose a reason for hiding this comment

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

혹시 createObjectURL사용도 고려해주실 수 있나요?

fileReader와 createObjectURL 비교

Copy link
Contributor Author

Choose a reason for hiding this comment

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

좋은 지적 감사합니다!
createObjectURL로 수정했습니다

@Banal972 Banal972 merged commit 9b39f3e into develop Aug 22, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 새로운 기능 추가 fix 오류를 수정함
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants