Fix/#123_marketAPI #153
Fix/#123_marketAPI #153
Conversation
| height: 124.spMin, | ||
| decoration: BoxDecoration( | ||
| borderRadius: BorderRadius.all(Radius.circular(10)), | ||
| image: marketBoard.imageUrls!.isEmpty ? null : DecorationImage( |
There was a problem hiding this comment.
๋ง์ง๋ง ๊ฒ์ํ์ด Bad States๊ฐ ๋์จ ์ด์ ๊ฐ ์ ๋๋ก ๋ ๊ฐ์ ๋ถ๋ฌ์ค์ง ๋ชปํ๋๋ฐ ์ฐธ์กฐํด์ ์๊ธด ์ค๋ฅ์ธ ์ค ์์๋๋ฐ ์ธ๋ฑ์ค์ ๋ฆฌ์คํธ ๊ธธ์ด ํ์ธํด๋ณด๋๊น ๋ค ๋ง๋๋ผ๊ตฌ์
ํน์ ์ ๋๋ก ๋ฐ์ง ์์ ๊ฐ์ด ์๋ ํ๊ณ ์ถ๋ ฅ์ผ๋ก ํ๋์ฉ ์ฐ์ด๋ณด๋๊น ๋งจ ๋ง์ง๋ง ๊ฒ์๊ธ์ ์์ ์ ์์ฑ๋ ๊ฑฐ๋ผ image๊ฐ ๋น์ด์์์ต๋๋ค!
image๊ฐ ๋น์ด์์ ๋์ ๋ํ ์ํธ์์ฉ์ด ์์ด์ Bad States ์ค๋ฅ๋ฅผ ๋ถ๋ฅธ ๊ฒ ๊ฐ์ต๋๋ค
๋ง์ผ์ ์ด๋ฏธ์ง๊ฐ ํ์์ง๋ง ํน์๋ผ๋ ๋ชจ๋ฅผ ์ํฉ์ ์ํด ์ด๋ฏธ์ง๊ฐ ๋น์ด์์ผ๋ฉด ์ฐธ์กฐํ์ง ์๋๋ก ํ์ต๋๋ค!
There was a problem hiding this comment.
ํ ์๊ฐ๋ณด๋ค ํ๋ฌดํ๋ ์ด์ ์ด๋ค์...
| } | ||
|
|
||
| getMoreBookmarksCounts(int page) async { | ||
| //๋ฐ์์์ ๊ต์ฒดํ๋ ๊ฒ์ด ์๋ ์ถ๊ฐ๋ฅผ ํด์ผํ๋ฏ๋ก addAll ์ฌ์ฉ | ||
| articleList!.addAll(await APIs.getMarketArticles(page)); |
There was a problem hiding this comment.
๋ฐ์์์ ๊ต์ฒดํ๋ ๊ฒ์ด ์๋ ์ถ๊ฐ๋ฅผ ํด์ผํ๋ฏ๋ก addAll ์ฌ์ฉ
| final bookmarkProvider = Provider.of<BookmarksProvider>(context, listen: false); | ||
| // +1๋ ํ์ด์ง ๋ถ๋งํฌ ๋ฆฌ์คํธ๋ฅผ ๋ฐ์์จ๋ค. | ||
| // ๋ฐ์์จ ๊ฒ์๊ธ ๋ฆฌ์คํธ๋ addAll๋ก ์ถ๊ฐ๋ ๊ฒ์ด๋ฏ๋ก ๋ถ๋งํฌ ๋ฆฌ์คํธ๋ addAll๋ก ์ถ๊ฐํด์ ์ ๋ฐ์ดํธํ๋ ๋ก์งํ์ | ||
| bookmarkProvider.getMoreBookmarksCounts(page); |
There was a problem hiding this comment.
๋ฐ์์จ ๊ฒ์๊ธ ๋ฆฌ์คํธ๋ addAll๋ก ์ถ๊ฐ๋ ๊ฒ์ด๋ฏ๋ก ๋ถ๋งํฌ ๋ฆฌ์คํธ๋ addAll๋ก ์ถ๊ฐํด์ ์ ๋ฐ์ดํธํ๋ ๋ก์งํ์
๐คจ Motivation
๐ Key Changes
๐ To Reviewers