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

전체게시판 오류 수정 #116

Merged
merged 7 commits into from
Aug 31, 2023
Merged

전체게시판 오류 수정 #116

merged 7 commits into from
Aug 31, 2023

Conversation

KIM-JUHYEON-79
Copy link

🤨 Motivation

🔑 Key Changes

  • 전체게시판 전체 글 조회
  • 전체게시판 검색 기능은 요청 코드는 짰으나 UI 가 결정안돼서 연결은 안했습니다!

🙏 To Reviewers

@KIM-JUHYEON-79 KIM-JUHYEON-79 self-assigned this Aug 30, 2023
@KIM-JUHYEON-79 KIM-JUHYEON-79 requested a review from urlotus August 30, 2023 13:32
],
),
actions: [
IconButton(onPressed: (){}, icon: Icon(Icons.notifications_none, size: 30.h,)),
Copy link
Contributor

Choose a reason for hiding this comment

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

아이콘 같은 거 사이즈를 .h로 해서 height 기준으로 하면 z플립 같은 height가 긴 화면에서는 아이콘이 엄청 커지는 경우가 있더라구요..!! 확인해봐야겠지만 혹시나해서 말씀드립니답

Copy link
Author

Choose a reason for hiding this comment

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

아 나중에 긴 화면에뮬레이터로 확인해봐야겠네요 감사합니다

final articles = await APIs.TotalArticles();
setState(() {
});
} catch (e) {
Copy link
Contributor

Choose a reason for hiding this comment

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

나중에 AT-C-002 같은 액세스 토큰 만료 에러 처리하는 부분도 추가되면 될 것 같습니다!!

@@ -27,6 +27,6 @@ subprojects {
project.evaluationDependsOn(':app')
}

task clean(type: Delete) {
tasks.register("clean", Delete) {
Copy link
Contributor

Choose a reason for hiding this comment

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

요거는 뭐가 수정된 건가요??

Copy link
Author

Choose a reason for hiding this comment

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

새 환경에서 받아오면서 오류가 생겨 오류 해결 중에 수정한 부분인거같습니다 아마 신경안쓰셔도 될거예요..!!

@KIM-JUHYEON-79 KIM-JUHYEON-79 merged commit 77d9e75 into develop Aug 31, 2023
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.

2 participants