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

[Fix] 본과정 시작 전 유저에 대한 "본과정 시작한지" 카드 오류 #296

Merged
merged 1 commit into from
Sep 24, 2023

Conversation

yoopark
Copy link
Contributor

@yoopark yoopark commented Sep 23, 2023

Summary

image

Describe your changes

음수로 표현하는 것은 UI 상 좋지 않아보이지만, 마땅히 생각나는 대체 문구가 없어서 일단 두었습니다.
추후 develop 브랜치에서 아직 들어오지 않은 신규 유저에 대한 고려를 별도 이슈로 올리겠습니다.

Issue number and link

@yoopark yoopark added the bug Something isn't working label Sep 23, 2023
@yoopark yoopark requested a review from 42sungwook September 23, 2023 03:11
@yoopark yoopark self-assigned this Sep 23, 2023
@jpham005
Copy link
Member

그냥 시작 전 (-11일) 이런식으로 쓰는건 어때요?

@yoopark
Copy link
Contributor Author

yoopark commented Sep 23, 2023

뭔가 그렇게 바꿀 바에는 본과정 시작한지라는 이름 자체를 바꿔야 더 자연스럽지 않을까 싶어서 고민입니다.

@jpham005
Copy link
Member

근데 -11 은 또 버그 같기도 해서요 ㅋㅋ

@yoopark
Copy link
Contributor Author

yoopark commented Sep 23, 2023

그럼 그냥 본과정 시작일을 표시하고, 더 이전에 들어온 사람에게 볼드 처리할까요?

@jpham005
Copy link
Member

그것보단 차라리 -11 일 이 나을 것 같습니다

@42sungwook
Copy link
Contributor

본 과정 시작한게 기준이면 음수는 다 0으로 바꿔도 괜찮을 것 같아요
근데 저도 그냥 - 붙여서 보여주는게 더 좋은 것 같습니다

@yoopark yoopark merged commit ff90308 into main Sep 24, 2023
@yoopark yoopark deleted the hotfix/v0.1.0 branch September 24, 2023 11:54
@42sungwook 42sungwook changed the title fix(versus): 본과정 시작 전 유저에 대한 "본과정 시작한지" 카드 오류 [Fix](versus): 본과정 시작 전 유저에 대한 "본과정 시작한지" 카드 오류 Oct 17, 2023
@42sungwook 42sungwook changed the title [Fix](versus): 본과정 시작 전 유저에 대한 "본과정 시작한지" 카드 오류 [Fix] 본과정 시작 전 유저에 대한 "본과정 시작한지" 카드 오류 Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants