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

[2.2.0] 쿠링봇 1차 QA 수정 #226

Merged
merged 4 commits into from
Aug 22, 2024
Merged

[2.2.0] 쿠링봇 1차 QA 수정 #226

merged 4 commits into from
Aug 22, 2024

Conversation

wonniiii
Copy link
Member

내용

  • 피그마에 올라온 쿠링봇 디자인 수정사항 적용했습니다. (폰트, 색상 수정)
  • 채팅 입력할 때마다 마지막 채팅으로 스크롤 되게 자동스크롤 구현했습니다.

참고사항

  • TCA 관련 코드 수정은 시간이 조금 길어질 것 같아서 일단 배포하고 차차 집중해서 수정하겠습니다!
  • 이전 PR들은 어떻게 할까요..?

@wonniiii
[수정] 1차 QA 반영
9311137 
*[수정] 색상 및 폰트 수정
@wonniiii
[수정] 1차 QA 반영
f7060db 
*[수정] 텍스트 복사 가능하게 수정
@wonniiii
[수정] ChatView 자동스크롤 구현
62d7c79
@wonniiii wonniiii added the 쿠링봇 쿠링봇 기능 label Aug 22, 2024
@wonniiii wonniiii requested review from lgvv and sunshiningsoo August 22, 2024 10:36
@wonniiii wonniiii self-assigned this Aug 22, 2024
@lgvv
Copy link
Member

lgvv commented Aug 22, 2024

  • 연관 PR 모두 머지 후에 QA 보면 조금 더 좋을 것 같습니다.
  • ios-map에 변경사항이 있었어서 해당 부분과 충돌나지 않는지도 확인해주세요!

sunshiningsoo
sunshiningsoo approved these changes Aug 22, 2024
View reviewed changes
Copy link
Member

@sunshiningsoo sunshiningsoo left a comment

Choose a reason for hiding this comment

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

앗 이미 머지되었군여
fatalError 던지는거 나중에 수정해주세요~~

긴작업 고생 많으셨습니다!

@lgvv
Copy link
Member

lgvv commented Aug 22, 2024

/쿠링 머지

@github-actions github-actions bot added the automerge auto merge 를 수동으로 미리 활성화 하는 경우 라벨을 추가해줘야 합니다. "/쿠링 머지" 실행시 자동으로 추가. label Aug 22, 2024
@github-actions github-actions bot merged commit ffb0415 into develop/2.0.0 Aug 22, 2024
@github-actions GitHub Actions
Copy link

🕊️ PR #226이 머지되었습니다.

@github-actions github-actions bot deleted the fix/kuringbot/QA branch August 22, 2024 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sunshiningsoo sunshiningsoo sunshiningsoo approved these changes

@lgvv lgvv Awaiting requested review from lgvv

Assignees

@wonniiii wonniiii

Labels
automerge auto merge 를 수동으로 미리 활성화 하는 경우 라벨을 추가해줘야 합니다. "/쿠링 머지" 실행시 자동으로 추가. 쿠링봇 쿠링봇 기능
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@wonniiii @lgvv @sunshiningsoo