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

자습실 신청 이름 오류 고치기 #245

Merged

Conversation

DSMInhyeKang
Copy link
Member

@DSMInhyeKang DSMInhyeKang commented Apr 9, 2023

개요

  • 자습실 신청 시 자리에 신청자 이름이 나타나지 않는 오류가 발생했습니다.

작업사항

  • SeatStatus 처리를 변경하고, 신청자 이름 미표기 오류를 해결했습니다

변경로직

seat.status == .available ? num : name
--> seat.student == nil? num : name

변경전

seat.status == .available ? num : name

변경후

seat.student == nil? num : name

기타

Comment로 논의해야할 게 있습니다

onAppear에 자습실 목록 가져오면 원래 400 => count 세서 맨 처음은 호출 X (== dropFirst()로 해야되는데 API request 부분에 넣었더니 작동 안 함)
@DSMInhyeKang DSMInhyeKang added 💬need discussion 논의가 필요한 경우 ♻️refactor 코드 리팩토링 할 경우 👾bug 버그가 발생한 경우 labels Apr 9, 2023
@DSMInhyeKang DSMInhyeKang self-assigned this Apr 9, 2023
@DSMInhyeKang DSMInhyeKang requested a review from a team as a code owner April 9, 2023 05:03
@DSMInhyeKang DSMInhyeKang linked an issue Apr 9, 2023 that may be closed by this pull request
@DSMInhyeKang
Copy link
Member Author

스크린샷 2023-04-09 오후 2 04 27

- 피그마 디자인

스크린샷 2023-04-09 오후 2 05 04

- 실제 앱

전에 취소 버튼이 disabled 된 것 같다는 이유로 컬러가 저렇게 바뀌었는데 신청 버튼처럼 disabled로 했다가 자리를 선택했을 때 풀리게 바꿀까요? @team-aliens/ios

@baekteun
Copy link
Member

baekteun commented Apr 9, 2023

이거 근데 단일 버튼 형식으로 가는거 아니였어요?

@baekteun
Copy link
Member

baekteun commented Apr 9, 2023

이거 근데 단일 버튼 형식으로 가는거 아니였어요?

취소 버튼이요?

둘이 통합해서요

@baekteun
Copy link
Member

baekteun commented Apr 9, 2023

이거 근데 단일 버튼 형식으로 가는거 아니였어요?

취소 버튼이요?

둘이 통합해서요

음...디자인엔 없었어요

스크린샷 2023-04-09 오후 2 38 17

이거.. 아닌가요?

@baekteun
Copy link
Member

baekteun commented Apr 9, 2023

이거 근데 단일 버튼 형식으로 가는거 아니였어요?

취소 버튼이요?

둘이 통합해서요

음...디자인엔 없었어요

스크린샷 2023-04-09 오후 2 38 17 이거.. 아닌가요?

5차에 있는 걸로 하는 게 아니었어요....? 저거 버튼 하나로 통일하자는 게 어디서 나온 의견인가여,,,,,,

엄.. 저도 정확한 시점은 기억 안나는데.. 언젠가에 저거가지고 뭐 말하지 않아어요..?

@DSMInhyeKang
Copy link
Member Author

기억이 없어요,,,,, 제가 없는 회의가 있었나요 hoxy,,,

@baekteun
Copy link
Member

baekteun commented Apr 9, 2023

기억이 없어요,,,,, 제가 없는 회의가 있었나요 hoxy,,,

예전에 막 준수님이 무지개 버튼 만들고 그런..

@baekteun
Copy link
Member

baekteun commented Apr 9, 2023

기억이 없어요,,,,, 제가 없는 회의가 있었나요 hoxy,,,

예전에 막 준수님이 무지개 버튼 만들고 그런..

음.. 안드도 2버튼으로 처리하네요. 이건 없어졌던거로..

@DSMInhyeKang DSMInhyeKang merged commit f46ef17 into develop Apr 9, 2023
@DSMInhyeKang DSMInhyeKang deleted the 244-studyroom-seat-applicant-name-does-not-appear branch April 9, 2023 23:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👾bug 버그가 발생한 경우 💬need discussion 논의가 필요한 경우 ♻️refactor 코드 리팩토링 할 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

자습실 신청 이름 오류 고치기
2 participants