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

195 decorate last item message #209

Merged
merged 6 commits into from
Aug 8, 2024
Merged

Conversation

sparcscasio
Copy link
Contributor

Overview

It closes #195

Changes

  • 마지막 화면 도달 시 출력되는 메시지 꾸미기..?
  • 번역 추가

Implementaion Method

After Changes

image

Related Issues

Rollback Scenario

TODO

thomaskim1130 and others added 6 commits May 27, 2024 15:53
마지막 페이지(게시물)에서 _loadNextPage()를 호출할 시, 따로 처리해 _isLastItem에 저장하도록 함.

Next Page가 api 상에서 존재하지 않을 시 CastError가 호출되는 것을 catch하고, 이후 LoadingIndicator()를 대신하여 표시됨.
_isLastItem 이 True일 때, context.locale 을 참조하여 메시지를 번역하도록 함.
postListShowPage에서 _loadNextPage()가 호출되었을 때, 로딩에 성공한다면 _isLastItem=false로 설정함. reloading 후에도 _isLastItem=true로 유지되는 현상을 해결함.
Resolved conflicts with origin/dev
@sparcscasio sparcscasio added the enhancement New feature or request label Aug 7, 2024
@sparcscasio sparcscasio added this to the v1.2.0 milestone Aug 7, 2024
@sparcscasio sparcscasio self-assigned this Aug 7, 2024
Copy link
Contributor

@thomaskim1130 thomaskim1130 left a comment

Choose a reason for hiding this comment

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

LGTM,
다음에는 branch 이름은 노션의 Ara/App/Git Conventions에 있는 규칙을 준수해주세요!

그리고 이번과 같이 기존 branch를 수정할 때에는, 꼭 다른 이슈와 branch를 생성하지 않고 기존 branch를 pull하여서 커밋만 하셔도 됩니다.

@thomaskim1130 thomaskim1130 merged commit 735d4b9 into dev Aug 8, 2024
2 checks passed
@thomaskim1130 thomaskim1130 deleted the 195-decorate-last-item-message branch August 8, 2024 04:28
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.

[Feature] PostListShowPage에서 마지막 페이지일 경우 알림
2 participants