Skip to content

[FE] 링크 공유 시 참여자 이름 넣기 #996

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

Merged
merged 3 commits into from
Feb 10, 2025
Merged

Conversation

jinhokim98
Copy link
Contributor

issue

구현 사항

링크 공유 시 참여자 이름이 보이도록 추가

image

링크 공유할 때 참여자 이름을 추가해서 쉽게 언급할 수 있도록 했습니다.

image url s3의 이미지로 교체

크루위키 문서 속 이미지를 가져오는 방식이 아닌, 우리 서비스 내에서 가져오도록 바꿨습니다.

🫡 참고사항

@github-actions github-actions bot requested review from pakxe, soi-ha and Todari February 5, 2025 09:09
@jinhokim98 jinhokim98 added ⚙️ feat feature 🖥️ FE Frontend labels Feb 5, 2025
@jinhokim98 jinhokim98 added this to the v3.2.1 milestone Feb 5, 2025
Copy link

github-actions bot commented Feb 5, 2025

const eventId = getEventIdByUrl();
const url = getEventPageUrlByEnvironment(eventId, 'home');

const shareInfo = {
title: `행동대장이 ${eventName}에\n대한 정산을 요청했어요 :)`,
title: `${allMembers.join(', ')}\n행동대장이 ${eventName}에 대한 정산을 요청했어요 :)`,
Copy link
Contributor

Choose a reason for hiding this comment

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

이름 출력하고 줄 바꿈 하고 한 줄 띄우고 행동대장 ~ 이 나오도록 해주실 수 있을까요?.?

그리고 이름 앞에 참여자같은 키워드를 넣어주는건 어떤가요?? 처음 이용하는 사용자는 저 이름이 뭘 뜻하는 건지 바로 인지하지 못할 수도 있을 것 같아서요!

ex)
참여자: 소하,웨디,쿠키,토다리

행동대장이 ~~~

Copy link
Contributor Author

@jinhokim98 jinhokim98 Feb 5, 2025

Choose a reason for hiding this comment

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

좋은 것 같아요! 이렇게 수정해서 올리겠습니당~

참여자: 소하, 웨디, 쿠키, 토다리

행동대장이 :crown: 행동대장 서비스 많관부 :purple_heart:에 대한 정산을 요청했어요 :)
아래 링크에 접속해서 정산 내역을 확인해 주세요!
https://dev.haengdong.pro/event/46d7ebff-fae3-47f9-8711-52040a105ca6/home

a699ecd

Copy link

github-actions bot commented Feb 5, 2025

Copy link
Contributor

@soi-ha soi-ha 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

@pakxe pakxe left a comment

Choose a reason for hiding this comment

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

오! 좋은거같습니다..!! 슬랙의 키워드 알람으로 받아볼 수 있으니까요 👍👍

@jinhokim98 jinhokim98 merged commit 7a28f86 into fe-dev Feb 10, 2025
2 checks passed
@jinhokim98 jinhokim98 deleted the feature/#995 branch February 10, 2025 05:54
@soi-ha soi-ha mentioned this pull request Feb 10, 2025
soi-ha pushed a commit that referenced this pull request Feb 10, 2025
* feat: 공유 링크에 참여자 이름을 추가하는 기능 추가

* refactor: 카카오톡 image url getImageUrl을 사용하여 가져오도록 설정

* feat: 참여자 명단 앞에 참여자 키워드 붙이고, 참여자 뒤 개행 추가
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

3 participants