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.

Sign up for GitHub

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

Jump to bottom

[Fix] 세부 내역 방 버그 수정 #1317 #1329

Merged
merged 39 commits into from
Apr 1, 2024
Merged

[Fix] 세부 내역 방 버그 수정 #1317 #1329

merged 39 commits into from
Apr 1, 2024

Conversation

contemplation-person
Copy link
Contributor

📌 개요

  • 버그 개선 및 CSS 수정

💻 작업사항

  • [Feat] 토스트바로 슬랙 초대 알림 메시지 전송 띄움

  • [Feat] 댓글 스크롤하도록 변경, 컴포넌트 이름 변경

  • [Feat] 댓글, 방내역, 인원이 많아지면 스크롤

  • [Feat] 모집인원에 최소인원 추가

  • [Style] report 모달 이름 변경.

  • [Refector] 쓰지 않는 함수 지움

✅ 변경로직

  • [Fix] 공유 주소 수정
  • [Fix] 프로필 컬러, createbar 로직 수정
  • [Fix] 타입 수정 intraid -> intraId
  • [Fix] 신고가 정상적으로 입력되지 않는 문제 해결

@contemplation-person
[Fix] 공유 주소 수정
76410bf
@contemplation-person
[Fix] 프로필 컬러, createbar 로직 수정
4ee3747 
프로필 컬러가 완전 어두운 색은 나오지 않도록 컬러 변경.
유저가 방에 참여하지 않았음에도 createbar가 나오는 버그 수정.
@contemplation-person
[Feat] 모집인원에 최소인원 추가
613897c
@contemplation-person
[Style] report 모달 이름 변경.
163e5fa
@contemplation-person
[Refector] 쓰지 않는 함수 지움
9a71872
@contemplation-person
[Fix] 신고가 정상적으로 입력되지 않는 문제 해결
dc6ab4a
@contemplation-person
[Feat] 토스트바로 슬랙 초대 알림 메시지 전송 띄움
1124075
@contemplation-person
Merge branch '6th_party' of github.com:42organization/42gg.client int…
f00f3c6 
…o Fix/세부-내역-방-버그-#1317
@contemplation-person
[Feat] 댓글 스크롤하도록 변경, 컴포넌트 이름 변경
4c029f6
@contemplation-person
[Feat] 댓글, 방내역, 인원이 많아지면 스크롤
e3c0254
@contemplation-person
[Fix] 타입 수정 intraid -> intraId
9e97bfe
@contemplation-person contemplation-person added bug Something isn't working user 일반 사용자가 사용하는 서비스에서 발생한 issue labels Mar 26, 2024
@contemplation-person contemplation-person self-assigned this Mar 26, 2024
izone00
izone00 reviewed Mar 27, 2024
View reviewed changes
Copy link
Contributor

@izone00 izone00 left a comment

Choose a reason for hiding this comment

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

디자인을 수정하면 사진 올려주세요
코드만 보고 뭐가 변했는지 상상하는건 너무 힘들어요

@contemplation-person
Copy link
Contributor Author

Screen Shot 2024-03-27 at 6 15 16 PM

@contemplation-person
Copy link
Contributor Author

image

  • 전체적인 디자인
  • 수정사항, 펼치기 접기 기능 넣기.

@contemplation-person
Copy link
Contributor Author

[ 댓글 접기 화면 ]
스크린샷 2024-03-29 오후 4 54 05

[댓글 펼치기 화면]
스크린샷 2024-03-29 오후 4 54 18

@contemplation-person contemplation-person force-pushed the Fix/세부-내역-방-버그-#1317 branch from 8587aef to 339b90f Compare March 29, 2024 09:57
@contemplation-person contemplation-person force-pushed the Fix/세부-내역-방-버그-#1317 branch from 0b5239c to c32cb7a Compare March 29, 2024 21:33
@contemplation-person contemplation-person force-pushed the Fix/세부-내역-방-버그-#1317 branch from 9c73d3a to 1171324 Compare March 29, 2024 23:06
@contemplation-person
Copy link
Contributor Author

스크린샷 2024-03-30 080923
노쇼 버튼 추가

@contemplation-person
[Fix] css 에러 해결
d38ad57 
경고의 원인:
  CSS에서 justify-content: start; 또는 align-items: start;와 같이 start 값을 사용했을 때,
  모든 브라우저에서 일관된 지원을 받지 못할 수 있습니다.
  이로 인해 autoprefixer가 경고를 발생시킵니다.

해결 방법:
  justify-content: start;를 justify-content: flex-start;로 변경하세요.
  align-items: start;를 align-items: flex-start;로 변경하세요.
@contemplation-person
[Fix] map에서 key값이 없을 때 발생하는 오류 수정
83137d6 
react는 여러개의 리스트를 생성할 때 key값으로 구분하여 성능을 향상시킨다.
현재 에러는 <></>로 감싸진 map이 있기 때문에 에러가 발생한다.
@contemplation-person
[Fix] comment 인자를 안넘겨주는 실수를 함.
382bc05
@contemplation-person
[Fix] 노쇼신고 버그 수정
5f75db0
@contemplation-person
[Fix] 에러로직 추가
4bfae57
@contemplation-person contemplation-person merged commit 8b695bf into 6th_party Apr 1, 2024
@contemplation-person contemplation-person deleted the Fix/세부-내역-방-버그-#1317 branch April 1, 2024 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@izone00 izone00 Awaiting requested review from izone00

@mantoing mantoing Awaiting requested review from mantoing

Assignees

@contemplation-person contemplation-person

Labels
bug Something isn't working user 일반 사용자가 사용하는 서비스에서 발생한 issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@contemplation-person @mantoing @izone00