Skip to content

Conversation

@nidor022
Copy link
Collaborator

@nidor022 nidor022 commented Aug 14, 2025

📝 작업 내용

pr수정사항에서 빼먹은거 있어서 다시 수정해서 올립니다.

📷 스크린샷 (선택)

image
const setter =
      mode === "popular" ? setPopularCurrentPage : setRecentCurrentPage;

이부분 자동 줄바꿈되는데 이유는 알수없음...

…게 구현

test데이터도 아직 완벽하게 테스트용은 아니고 단지 출력이 잘 되는지만 테스트할려고 뽑은거라서 아~~~주 간단하게만 해놓았습니다.
…게 구현

test데이터도 아직 완벽하게 테스트용은 아니고 단지 출력이 잘 되는지만 테스트할려고 뽑은거라서 아~~~주 간단하게만 해놓았습니다.
pr수정사항 적용하였고, 페이지랑 내부 컴포넌트 분리할 src\features\rolling-paper\components 파일경로 추가하였습니다.
@nidor022 nidor022 self-assigned this Aug 14, 2025
@nidor022 nidor022 added this to Rolling Aug 14, 2025
@nidor022 nidor022 linked an issue Aug 14, 2025 that may be closed by this pull request
2 tasks
@cskime
Copy link
Collaborator

cskime commented Aug 14, 2025

이거 #28 PR에 들어가야할 commit들이면, PR 새로 만드는것보다 #28 에 이어서 push하는게 낫지 않나요?

@nidor022
Copy link
Collaborator Author

이거 #28 PR에 들어가야할 commit들이면, PR 새로 만드는것보다 #28 에 이어서 push하는게 낫지 않나요?

아 아까 #28에 이어서 하는데 알림이 안가길래 뭔가 잘못했나해서 새로 만들어서 올렸는데 나중에 확인해보니까 #28에 이어서 올라가있더라고여...

@nidor022 nidor022 changed the title fix/#37 [#37] fix #28수정사항에서 누락된것 추가 Aug 14, 2025
@cskime
Copy link
Collaborator

cskime commented Aug 14, 2025

이거 #28 PR에 들어가야할 commit들이면, PR 새로 만드는것보다 #28 에 이어서 push하는게 낫지 않나요?

아 아까 #28에 이어서 하는데 알림이 안가길래 뭔가 잘못했나해서 새로 만들어서 올렸는데 나중에 확인해보니까 #28에 이어서 올라가있더라고여...

#28 PR이랑 다른게 없다면, #28 에 수정하신게 모두 잘 올라가있는지 확인하시고 이 PR은 close 해주세요 🙏🏻

@nidor022 nidor022 closed this Aug 14, 2025
@github-project-automation github-project-automation bot moved this to Done in Rolling Aug 14, 2025
@cskime cskime removed this from Rolling Aug 16, 2025
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.

#27에서 수정사항 빠진부분 추가

2 participants