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

리팩토링 & 캘린더 수정 #31

Merged
merged 2 commits into from
Jun 17, 2021
Merged

리팩토링 & 캘린더 수정 #31

merged 2 commits into from
Jun 17, 2021

Conversation

qwa310
Copy link
Owner

@qwa310 qwa310 commented Jun 17, 2021

  • routes.dart : route 정보만 모아둠
  • screens.dart : screen 정보만 모아둠 - route.dart 와 연결됨
  • constants.dart : 중복 많은 UI / List 모아둠 : 쓸 때 변수 가져와서 사용
  • lib내에 있던 파일들을 screen / api 폴더로 구분 : 기존 파일들이 screen 단위로 구현된 거기 때문에 widget 단위로 쪼개는 일은 시간을 두고 진행할 예정
  • 파일명 변경
    • ~~view로 끝나는 파일 -> ~~screen 로 파일명 변경
    • 파이어베이스 연동하는데 쓰인 파일들은 ~api 로 파일명 변경
    • HomePage() -> MainScreen() 로 변경
  • 캘린더 및 월별 전력소비량 보여주는 화면에서 스와핑 할 때 1~12월까지만 바뀌게 로직 수정. -> month 정보로 월별로 사용자가 입력한 전기 사용량 리스트만 보여주면 될듯!!

@qwa310 qwa310 requested a review from hoit1302 June 17, 2021 23:25
@qwa310 qwa310 merged commit 7cc194d into master Jun 17, 2021
@qwa310 qwa310 linked an issue Jun 18, 2021 that may be closed by this pull request
9 tasks
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.

Refactoring and Update
2 participants