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

3주차- 정진용 #8

Merged
merged 1 commit into from
Nov 18, 2024
Merged

3주차- 정진용 #8

merged 1 commit into from
Nov 18, 2024

Conversation

JeongJinyong
Copy link
Collaborator

정진용 - 3주차 🚀

이번 주에는 '가슴속삼천원' 프로젝트에서 다음과 같은 버그 수정을 진행했습니다. 요즘 너무 쌓아놔서 아주 빡센 집중력으로 처리해버렸습니다. 엣헴

  • 가게 리스트 광고가 없는 경우 가게가 안보이는 버그 수정
  • 마이페이지 칭호 변경 시, 메달 이미지 잘림 수정
  • 가게 최근 등록순 조회시, 최대 거리 제한 없음 수정
  • 가게 상세화면 리뷰수/이미지 수 안맞는 문제 수정
  • 즐겨찾기 리스트 전체 삭제 후 복귀시 그대로 있는 문제 수정
  • 마이페이지 이동시 간헐적으로 즐겨찾기가 안보이는 문제 수정
  • 일부 API들 User-Agent가 비정상적으로 오는 현상 수정
  • 로그인 토큰 문제 수정
  • 마수동 OFF시 알럿 추가
  • 인기 가게 동네 밑에 짤림 수정

Kotlin 스터디 - DSL

이번 주 코틀린 스터디에서는 DSL(Domain Specific Language)에 대해 공부했습니다. DSL은 특정 문제 영역에 대해 간결하고 이해하기 쉽게 작성된 프로그래밍 언어입니다. 안드로이드 개발자 입장에서 Gradle 스크립트를 작성할 때 많이 접하게 되는 개념이죠. 근데 사실 책에선 그래들에 대한 내용이 붙어서 나오지 않고 HTML 사용과 연결되어서 설명합니다. 그래서 해당부분에 대해서 이해하기 어려웠고 블로그나 GPT를 통해서 이해하고자 노력하였습니다. 그래서 결국 이번 스터디를 통해 DSL이 코틀린만의 개념이 아니라, 프로그래밍 전반에서 사용되는 개념이라는 것을 이해하게 되었습니다.

이번 주 회고

이번 주는 '가슴속삼천원' 프로젝트의 버그 수정을 통해 사용자들이 겪던 문제를 해결했고, DSL에 대해 공부하면서 코드의 가독성과 사용성을 높일 수 있는 방법을 배웠습니다. DSL관련해서 다음주에 글또 블로그를 작성해보고자 합니다. 원래 계획은 리플렉션인데 DSL개념이 좀 더 쉬울거라고 생각해서 차근차근 쌓아올리는 순서로 가고자합니다.

@JeongJinyong JeongJinyong self-assigned this Nov 15, 2024
Copy link

PR 본문이 300자 이상입니다. ✔️ PR 조건을 충족했습니다.

@Leesin0222
Copy link
Member

Leesin0222 commented Nov 16, 2024

이번 주 프젝에 집중을 빡! 하셨나보군요...
그런 집중력이 너무너무 부럽습니다

이제 곧 겨울이 다가오는데 가슴속3천원 다운받으러 가야겠군요

@Gyeony95
Copy link
Member

고생하셨습니다 ㅎㅎ 가슴속삼천원의 역사에대해서 사이데이션때 들어보고싶네요 ㅋㅋㅋ

@JeongJinyong JeongJinyong merged commit ac911bd into main Nov 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants