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 : 프로필 / 포스트디테일 배포 발견 오류 해결 #136

Merged
merged 14 commits into from
Sep 27, 2023

Conversation

hyesung99
Copy link
Collaborator

🪄 변경사항

  1. 댓글 삭제 버튼 닉네임 오른쪽에 위치하기
  2. 내용을 지우고 빈상태로 저장해버리면 명상글이 나오지 않음 => 빈 상태로 제출하면 토스트 띄움
  3. 팔로잉, 팔로우버튼 헷갈리는 문제 => 정말 어렵게 해결
  4. DM 아이콘 삭제
  5. 댓글 길이 제한 두기 => 500자 제한, 넘으면 토스트 띄움
  6. 닉네임변경 시 닉네임 있을때만 변경요청
  7. 댓글 purify하기
  8. 남의 계정일때 팔로잉 목록에서 내 기준 팔로우, 팔로워 요청 => 진짜 머리 터지는 줄 followUsers에서 내 유저 정보 불러와서 팔로우 데이터와 비교하여 팔로우했는지 확
  9. 게시글 수정 취소하면 원문 복구하기

🖥 결과 화면

image

✏️ PR 포인트

아직 팔로우 요청을 하면 알 수 없는 에러가 납니다.
그런데 정상작동 합니다ㅠ.ㅠ
아직 에러를 잡을 마땅한 방법이 떠오르지 않아 일단 PR합니다

Copy link
Collaborator

@nayeon-hub nayeon-hub left a comment

Choose a reason for hiding this comment

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

Look good to me🔥

팔로우 팔로워 머리 많이 아프셨을텐데 진짜 수고 많으셨습니다🙇‍♀️

Copy link
Member

@sscoderati sscoderati left a comment

Choose a reason for hiding this comment

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

Good~ 👍

onSuccess: (res) => {
postNotifications(token, {
notificationType: 'COMMENT',
notificationTypeId: res._id,
userId: userId,
postId: res.post
});
refetch();
}
Copy link
Member

Choose a reason for hiding this comment

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

onError 처리는 없어도 괜찮을까요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

onError처리 해야하는데 말이죠... 시간이 참 없네요ㅠㅠ
마감 후에라도 하겠습니다...!

@suyeon1218 suyeon1218 merged commit 3dadc0f into main Sep 27, 2023
@suyeon1218 suyeon1218 deleted the fix/profile/deployError branch September 27, 2023 03:27
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.

4 participants