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

[feat] 한경대 서비스 관련 추가 구현 사항점 #260

Merged
merged 11 commits into from
Sep 18, 2024

Conversation

wjdtkdgns777
Copy link
Collaborator

@wjdtkdgns777 wjdtkdgns777 commented Sep 17, 2024

화면 캡처 2024-09-17 201528

  1. 운영시간 관련하여 추가적인 처리가 이루어졌습니다(오픈시간, 닫는시간, 운영중인지 여부)
  2. 현재 남은 수량 재고 관련 구현 및 품절 시 흐릿하게 + 품절이라고 뜨도록 표시
  3. 부스 상세화면 진입시 내 웨이팅 리스트를 가져와, 그정보를 토대로 이 부스에 이미 웨이팅이 되어있는지, 웨이팅 개수가 이미 3개를 걸어두었는지 판별, 웨이팅 버튼을 누르면 스낵바로 해당사항들을 보여줍니다

화면 캡처 2024-09-17 201839

  1. 웨이팅 부재로 취소되었을시(노쇼) 추가를 처리해주었습니다

문제점이 있는데 웨이팅리스트에서 유니페스트 아웃라인 버튼이 색이 안먹고 핑크빛이 돕니다. 이런게 전에도 있었는데, 그때는 유니페스트 테마 덧씌워주면서 해결하셨던걸로 알아 시도해봤는데 안되길래(훗날 그냥 maxsize로 해결보신거 같긴 합니다만) 여러가지 시도했는데 안되어서.., 혹시 요부분 아실까요? 그리고 이런게 바텀시트에 학교들도 그렇더라구요. 핑크빛이 좀 도는..
이외에도 테스트 한번 해봐야겠어요.

  1. 한경대로 지도를 다시 잡았습니다

이제 해야할것

  1. 클러스터링
  2. 에러잡기 및 자연스러운 flow생각해보기? 정도 있겠네요

@wjdtkdgns777 wjdtkdgns777 added the enhancement New feature or request label Sep 17, 2024
@wjdtkdgns777 wjdtkdgns777 self-assigned this Sep 17, 2024
Copy link
Collaborator

@easyhooon easyhooon left a comment

Choose a reason for hiding this comment

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

리뷰한 부분 해결 후 merge 하시면 될 것 같습니다. 수고하셨습니당

@easyhooon
Copy link
Collaborator

클러스터링 관련 부분은 물리적으로 불가능한 부분이 있어서 현재 클러스터링을 활성화하면 어떻게 보이는지와 기존을 비교해서 논의를 통해 둘중 어떤것을 선택하면 될지 결정하면 좋을 것 같습니다

wjdtkdgns777 and others added 2 commits September 18, 2024 18:09
개발의 생산성을 높히기 위함, 하드 코딩된 문자열 string resource 추출
@easyhooon
Copy link
Collaborator

[refactor] resources 각 모듈별로 분리

droidknights, nia 의 방식과 같은 방식으로 각 모듈에서 참조하는 resources 들은 각 모듈 내에서 정의하는 방식으로 변경하였습니다.(제가 해당 레포들의 모듈화 구성 방식 잘못이해하고 있었음...)

이제는 string resource 를 추출할때 해당 모듈의 res 폴더내에 바로바로 string resource 들을 만들수있기 때문에 개발에서도 생산성이 올라갈것으로 예상됩니다. (이전엔 designsystem 모듈내에 res 파일에 해당 string resource 를 정의하고 이것을 참조하도록 코드를 작성해줘야했음)

@wjdtkdgns777 wjdtkdgns777 merged commit 25bfe5e into develop Sep 18, 2024
1 check passed
@wjdtkdgns777 wjdtkdgns777 deleted the feat/QA branch September 18, 2024 15:27
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