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

[#9] 키워드 입력값 파라미터 추가 #10

Merged
merged 6 commits into from
Jan 8, 2025

Conversation

myoungji-kim
Copy link
Collaborator

@myoungji-kim myoungji-kim commented Jan 8, 2025

이슈

키워드 별로 남은 시간을 계산할 수 있으면 더 활용도가 좋지 않을까? 라는 생각에 파라미터를 하나 더 추가해보았습니당
사실 "점심시간 얼마 남았어요"도 쓰고 싶었다는 건 안 비밀😸

AS-IS

입력값 종류:
{시간}

TO-BE

입력값 종류:
{키워드} {시간}

리뷰 요구사항

코틀린 찍먹 경험 밖에 없기에.. 간단하게만 뚝딱뚝딱 해보았습니당! >> 피드백 대환영!!
키워드 아이디어는 혼자 똑딱 생각나서 추가해본거라서, 다른 의견들도 아주 자유롭게 받습니당!! 편하게 리뷰 주세여 ✋🏼

@easternkite easternkite linked an issue Jan 8, 2025 that may be closed by this pull request
@myoungji-kim myoungji-kim self-assigned this Jan 8, 2025
@myoungji-kim myoungji-kim added the enhancement New feature or request label Jan 8, 2025
Copy link
Collaborator

@easternkite easternkite left a comment

Choose a reason for hiding this comment

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

기존 명령어에 키워드라는 개념을 추가하여
더욱 유연한 리스폰스를 받을 수 있도록 잘 작성해주셨네요~ 👍

다만 기존 포맷이었던 /time {시간}은 사용이 불가능해진것으로 보이는군요!
간편하게 /time {시간} 포맷을 원하시는 분도 계실 것 같아서,
기존 포맷도 함께 사용할 수 있도록 구현되면 좋을 것 같아요~~

선택지 1. /time {시간} -> 퇴근시간까지 9시간 40분 남았습니다 😂 
선택지 2. /time {키워드} {시간} -> {키워드} 시간까지 9시간 40분 남았습니다 😂 

@myoungji-kim myoungji-kim force-pushed the feature/add-keyword-input branch from 4101a11 to fb975a9 Compare January 8, 2025 15:56
@myoungji-kim
Copy link
Collaborator Author

기존 명령어에 키워드라는 개념을 추가하여 더욱 유연한 리스폰스를 받을 수 있도록 잘 작성해주셨네요~ 👍

다만 기존 포맷이었던 /time {시간}은 사용이 불가능해진것으로 보이는군요! 간편하게 /time {시간} 포맷을 원하시는 분도 계실 것 같아서, 기존 포맷도 함께 사용할 수 있도록 구현되면 좋을 것 같아요~~

선택지 1. /time {시간} -> 퇴근시간까지 9시간 40분 남았습니다 😂 
선택지 2. /time {키워드} {시간} -> {키워드} 시간까지 9시간 40분 남았습니다 😂 

@easternkite
안그래도 그 부분이 마음에 걸렸는데, 좋은 의견 감사합니당 💯 💯

📌 fb975a9
말씀주신 내용대로 기존 방식도 지원하고, 원하는 키워드가 있을 경우 이어서 추가로 입력받도록 하는 방향이 더 간편하다고 생각이 들어서
파라미터 입력값 순서 변경 및 기존 방식 대응되도록 변경하였는데욥! 한번더 확인 부탁드립니다~!

Copy link
Collaborator

@easternkite easternkite left a comment

Choose a reason for hiding this comment

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

안그래도 그 부분이 마음에 걸렸는데, 좋은 의견 감사합니당 💯 💯
📌 fb975a9
말씀주신 내용대로 기존 방식도 지원하고, 원하는 키워드가 있을 경우 이어서 추가로 입력받도록 하는 방향이 더 간편하다고 생각이 들어서
파라미터 입력값 순서 변경 및 기존 방식 대응되도록 변경하였는데욥! 한번더 확인 부탁드립니다~!

@myoungji-kim 모든 리비전 확인 완료했습니다.
고생하셨습니다!

@myoungji-kim
Copy link
Collaborator Author

드디어 main에 첫 커밋...!!! ㄷㄱㄷㄱ!!!!!!!

@myoungji-kim myoungji-kim merged commit 2c339a4 into euuungb:main Jan 8, 2025
1 check passed
@myoungji-kim myoungji-kim deleted the feature/add-keyword-input branch January 8, 2025 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

키워드 입력값 파라미터 추가
2 participants