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

[QA] 웨이팅 부재 다이얼로그 처리, 새로고침에 딜레이 추가 #282

Merged
merged 11 commits into from
Oct 25, 2024

Conversation

wjdtkdgns777
Copy link
Collaborator

@wjdtkdgns777 wjdtkdgns777 commented Oct 25, 2024

웨이팅부분을 체크해서 부재처리된거면 처리하라는 다이얼로그가 뜨도록 처리했습니다
또 새로고침에 딜레이를 추가했는데 이 방식 말씀하신게 맞을까요?
그리고.. 왜인지 pulltorefresh에 코드를... 잘못 작성한 부분이 있었습니다 발견해서 다행인데 뭔짓을 했던거지...

@wjdtkdgns777 wjdtkdgns777 added the enhancement New feature or request label Oct 25, 2024
@wjdtkdgns777 wjdtkdgns777 self-assigned this Oct 25, 2024
@wjdtkdgns777 wjdtkdgns777 changed the title [QA] 웨이팅 부재 토스트메시지 추가, 새로고침에 딜레이 추가 [QA] 웨이팅 부재 다이얼로그 처리, 새로고침에 딜레이 추가 Oct 25, 2024
@easyhooon
Copy link
Collaborator

웨이팅 화면에 로딩이 지금 하나도 없네요 LoadingWheel 보이게 설정 해야할것같아여

@easyhooon
Copy link
Collaborator

그리고 init 할때는 delay(1000) 필요 없을거같거든요 init 이랑 refresh 일때 케이스 나눠서 refresh 일때만 delay 걸리게 해주는게 좋을 것 같슴다

@easyhooon
Copy link
Collaborator

image
문구가 변경될 예정이기 한데, 이렇게 description text 가 길어지면 dialog 전체 column 혹은 text 양쪽에 padding 넣어야 할 것 같아요
두 문장을 \n 로 쪼개서 한줄씩 배치해도 좋을 것 같구요

@wjdtkdgns777
Copy link
Collaborator Author

image 문구가 변경될 예정이기 한데, 이렇게 description text 가 길어지면 dialog 전체 column 혹은 text 양쪽에 padding 넣어야 할 것 같아요 두 문장을 \n 로 쪼개서 한줄씩 배치해도 좋을 것 같구요

요부분은 \n으로 쪼갰습니다!

@wjdtkdgns777
Copy link
Collaborator Author

훨 보기 좋아졌군..

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.

LGTM!

@wjdtkdgns777 wjdtkdgns777 merged commit ad217e9 into develop Oct 25, 2024
1 check passed
@wjdtkdgns777 wjdtkdgns777 deleted the qa/qa branch October 25, 2024 15:23
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