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

[chat] 답장하기 버튼에 data-id 추가 #3556

Merged
merged 3 commits into from
Jan 23, 2025
Merged

Conversation

guswl98
Copy link
Contributor

@guswl98 guswl98 commented Jan 22, 2025

PR 설명

reply button을 구분하기 위해 data-id를 추가합니다. chat-web에서 https://github.com/titicacadev/triple-chat-web/pull/397 이렇게 사용되고 있습니다.

변경 내역

  • ReplyActionButtondata-id를 추가합니다.
  • ReplyActionButton에 사용되는 data-id를 외부에서 사용할 수 있도록 constants에 추가합니다.

@guswl98 guswl98 self-assigned this Jan 22, 2025
Copy link

codecov bot commented Jan 22, 2025

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 10.10%. Comparing base (560355c) to head (d6e0f94).
Report is 3 commits behind head on epic/geochat.

Files with missing lines Patch % Lines
packages/chat/src/bubble-container/bubble-info.tsx 0.00% 3 Missing ⚠️
packages/chat/src/chat/constants.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff                @@
##           epic/geochat    #3556      +/-   ##
================================================
- Coverage         10.10%   10.10%   -0.01%     
================================================
  Files               726      726              
  Lines             28220    28221       +1     
  Branches           7720     7720              
================================================
  Hits               2852     2852              
- Misses            25348    25349       +1     
  Partials             20       20              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@guswl98
Copy link
Contributor Author

guswl98 commented Jan 22, 2025

release-canary

Copy link

v13.34.1-pr-3556.53 has been published!

@guswl98
Copy link
Contributor Author

guswl98 commented Jan 22, 2025

release-canary

Copy link

v13.34.1-pr-3556.55 has been published!

@guswl98
Copy link
Contributor Author

guswl98 commented Jan 22, 2025

release-canary

Copy link

v13.34.1-pr-3556.56 has been published!

@guswl98 guswl98 requested a review from dongoc January 23, 2025 02:28
@guswl98 guswl98 marked this pull request as ready for review January 23, 2025 02:28
@guswl98 guswl98 requested a review from a team as a code owner January 23, 2025 02:28
@guswl98 guswl98 requested review from gigibean and zhsks528 and removed request for a team January 23, 2025 02:28
@@ -3,7 +3,8 @@ import { Container, Text } from '@titicaca/core-elements'
import { format, setDefaultOptions } from 'date-fns'
import { ko } from 'date-fns/locale'

import { ReplyMessageIcon } from '../icons/reply-meesage-icon'
import { ReplyMessageIcon } from '../icons/reply-message-icon'
Copy link
Contributor

Choose a reason for hiding this comment

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

앜ㅋㅋㅋ 파일명에 오타가 있었군요

@guswl98 guswl98 merged commit 7d73290 into epic/geochat Jan 23, 2025
16 checks passed
@guswl98 guswl98 deleted the feat/add-id branch January 23, 2025 05:17
@dongoc dongoc mentioned this pull request Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants