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.

Sign up for GitHub

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

Jump to bottom

[AN] LiveData to Flow 리팩터링 (티켓 제시, 회원 가입) #494

Closed
re4rk opened this issue Oct 4, 2023 · 0 comments · Fixed by #498
Closed

[AN] LiveData to Flow 리팩터링 (티켓 제시, 회원 가입) #494

re4rk opened this issue Oct 4, 2023 · 0 comments · Fixed by #498
Assignees
@re4rk
Labels
AN 안드로이드에 관련된 작업 USER ⚙️ 리팩터링 리팩터링에 관련된 작업
Milestone
6차 데모데이

Comments

@re4rk
Copy link
Collaborator

re4rk commented Oct 4, 2023

✨ 세부 내용

  • TicketEntry, SignIn 화면 리팩토링

⏰ 예상 소요 시간

  • 6시간
@re4rk re4rk added AN 안드로이드에 관련된 작업 USER ⚙️ 리팩터링 리팩터링에 관련된 작업 labels Oct 4, 2023
@re4rk re4rk self-assigned this Oct 4, 2023
@re4rk re4rk removed this from 2023-festa-go Oct 4, 2023
@re4rk re4rk added this to the 6차 데모데이 milestone Oct 4, 2023
@re4rk re4rk linked a pull request Oct 5, 2023 that will close this issue
[AN/USER] refactor: LiveData to Flow 리팩터링 (티켓 제시, 로그인 화면) (#494) #498
Merged
@re4rk re4rk mentioned this issue Oct 5, 2023
Merged
SeongHoonC pushed a commit that referenced this issue Oct 7, 2023
@re4rk
[AN/USER] refactor: LiveData to Flow 리팩터링 (티켓 제시, 로그인 화면) (#494) (#498)
31cfe5e 
* refactor: LiveData에서 UiState로 변경전에 Legacy로 변경

* refactor: LiveData에서 StateFlow로 변경

* refactor: 리포지터리 플로우로  변경

* refactor: combine으로 데이터를 입력을 받도록 변경

* refactor: Flow 래핑한 Repository를 다시 래핑해제

* refactor: loadTicket에서 TicketCode까지 받아오는 기능 제거

* refaactor: 티켓 입장 및 테스트 케이스 변경

* refactor: 로그인 화면 플로우로 변경

* refactor: LiveData를 위한 규칙 제거

* refactor: 중복되는 viewModelScope 제거

* refactor: 내부 함수 private 적용

* refactor: 파타미터 네이밍을 명시적으로 변경

* refactor: 리베이스 후 생긴 버그 수정

* test: 테스트 given절 네이밍 수정

* test: given절 용도에 맞는 함수로 변경

* test: 로그인 화면 테스트코드 수정
@github-project-automation github-project-automation bot moved this from Todo to Done in 2023-festa-go Oct 7, 2023
BGuga pushed a commit that referenced this issue Oct 17, 2023
@re4rk @BGuga
[AN/USER] refactor: LiveData to Flow 리팩터링 (티켓 제시, 로그인 화면) (#494) (#498)
66efffc 
* refactor: LiveData에서 UiState로 변경전에 Legacy로 변경

* refactor: LiveData에서 StateFlow로 변경

* refactor: 리포지터리 플로우로  변경

* refactor: combine으로 데이터를 입력을 받도록 변경

* refactor: Flow 래핑한 Repository를 다시 래핑해제

* refactor: loadTicket에서 TicketCode까지 받아오는 기능 제거

* refaactor: 티켓 입장 및 테스트 케이스 변경

* refactor: 로그인 화면 플로우로 변경

* refactor: LiveData를 위한 규칙 제거

* refactor: 중복되는 viewModelScope 제거

* refactor: 내부 함수 private 적용

* refactor: 파타미터 네이밍을 명시적으로 변경

* refactor: 리베이스 후 생긴 버그 수정

* test: 테스트 given절 네이밍 수정

* test: given절 용도에 맞는 함수로 변경

* test: 로그인 화면 테스트코드 수정
BGuga pushed a commit that referenced this issue Oct 17, 2023
@re4rk @BGuga
[AN/USER] refactor: LiveData to Flow 리팩터링 (티켓 제시, 로그인 화면) (#494) (#498)
1f9333f 
* refactor: LiveData에서 UiState로 변경전에 Legacy로 변경

* refactor: LiveData에서 StateFlow로 변경

* refactor: 리포지터리 플로우로  변경

* refactor: combine으로 데이터를 입력을 받도록 변경

* refactor: Flow 래핑한 Repository를 다시 래핑해제

* refactor: loadTicket에서 TicketCode까지 받아오는 기능 제거

* refaactor: 티켓 입장 및 테스트 케이스 변경

* refactor: 로그인 화면 플로우로 변경

* refactor: LiveData를 위한 규칙 제거

* refactor: 중복되는 viewModelScope 제거

* refactor: 내부 함수 private 적용

* refactor: 파타미터 네이밍을 명시적으로 변경

* refactor: 리베이스 후 생긴 버그 수정

* test: 테스트 given절 네이밍 수정

* test: given절 용도에 맞는 함수로 변경

* test: 로그인 화면 테스트코드 수정
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@re4rk re4rk

Labels
AN 안드로이드에 관련된 작업 USER ⚙️ 리팩터링 리팩터링에 관련된 작업
Projects
2023-festa-go
Status: Done
Milestone
6차 데모데이
1 participant
@re4rk