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

ui: 도메인명 변경에 따라 추억을 카테고리로 수정 #578 #579

Merged
merged 3 commits into from
Jan 12, 2025

Conversation

hxeyexn
Copy link
Contributor

@hxeyexn hxeyexn commented Dec 31, 2024

⭐️ Issue Number

🚩 Summary

  • 도메인명 변경에 따라 UI 상에 보여지는 추억이라는 단어를 카테고리로 수정했습니다.

🙂 To Reviewer

  • 추억이 카테고리로 변경되지 않은 부분이 있다면 알려주세요!

@hxeyexn hxeyexn added android We are android>< ui xml 작업 labels Dec 31, 2024
@hxeyexn hxeyexn added this to the sprint-7 milestone Dec 31, 2024
@hxeyexn hxeyexn self-assigned this Dec 31, 2024
@hxeyexn hxeyexn linked an issue Dec 31, 2024 that may be closed by this pull request
2 tasks
Copy link
Contributor

@Junyoung-WON Junyoung-WON left a comment

Choose a reason for hiding this comment

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

추억 -> 카테고리로 빠르게 잘 바꿔주셨네요!!
고생하셨습니다 해나! 😄
추억이여 안녕!

@@ -39,29 +39,29 @@
<string name="main_staccato_creation_btn_description">스타카토 기록하기로 이동</string>

// fragment_timeline
<string name="timeline_my_timeline">나의 추억들</string>
<string name="timeline_my_timeline">나의 카테고리들</string>
Copy link
Contributor

Choose a reason for hiding this comment

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

예전에 회의할 때 타임라인에서 보여주는 문구는 나의 추억들로 고정하자고 의견이 나왔던 것으로 기억하는데, 기억이 가물가물해서 회의록도 찾아봤지만 그런 내용이 없네요!
혹시 어떻게 결정했는지 기억나시나요...?? 🤔

Copy link
Contributor Author

@hxeyexn hxeyexn Jan 11, 2025

Choose a reason for hiding this comment

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

헉 이 부분은 나의 추억들로 고정하자고 했던 것 같아요!
근데 사용자가 "OO의 추억들"이라는 단어를 보고 헷갈릴 일은 없을까요...? 갑자기 문득 그런 생각이 드네용...

혹시 빙티랑 백엔드분들 생각은 어떠신가요? (@s6m1n , @BurningFalls , @linirini , @Ho-Tea )

Copy link
Member

Choose a reason for hiding this comment

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

타임라인에서 보여주는 문구는 나의 추억들로 고정하자고 의견이 나왔던 것으로 기억하는데

맞습니다. "ㅇㅇ의 카테고리들"은 명확하지만 조금 딱딱한 감이 있어 메인 화면인 만큼 "ㅇㅇ의 추억들"을 사용하자고 했었어요.
이 부분은 A/B 테스트를 해보면 좋을 것 같기도 하네요 🤔

Copy link
Contributor

@BurningFalls BurningFalls Jan 12, 2025

Choose a reason for hiding this comment

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

"OO의 추억들"로 하기로 했었습니다.

제가 지인들에게 물어봤을 때, "OO의 추억들"이라고 적혀있으면 추억들이 폴더로 묶여있다고 이해하는 경향을 보였습니다.
"추억들"로 적어놓아도 하나의 그룹을 추억으로 인식하는게 아니라 각각의 스타카토를 추억으로 생각하더라구요.
그래서 차라리 "OO의 스타카토들"이라고 적고 "카테고리"는 바텀시트 상으로는 개념으로만 존재해도 상관없지 않을까 하는 생각도 들기는 합니다. (물론 카테고리 추가/수정에서는 '카테고리 추가하기' '카테고리 생성하기'로 표시됩니다.)

다만, 일단은 원래 회의에서 결정난대로 "OO의 추억들"로 하는게 좋을 것 같아요!

Copy link
Contributor

Choose a reason for hiding this comment

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

A/B 테스트 하시죠!!!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

좋습니다! 그럼 우선 "OO의 추억들"로 해두고 A/B테스트 하시죠~

Copy link
Member

@s6m1n s6m1n left a comment

Choose a reason for hiding this comment

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

LGTM~ 수고하셨습니다!

@hxeyexn hxeyexn merged commit b0bceed into develop Jan 12, 2025
1 check passed
@hxeyexn hxeyexn deleted the feature/#578_replace_memory_with_category branch January 12, 2025 05:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android We are android>< ui xml 작업
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

ui: UI 상에 보여지는 추억을 카테고리로 수정
5 participants