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

Fix: 개행으로 메시지가 전송되는 버그와 더 보기 버튼이 작동하지 않는 문제 수정 #568

Merged
merged 6 commits into from
Feb 14, 2025

Conversation

catsmiry
Copy link
Contributor

@catsmiry catsmiry commented Feb 12, 2025

What

수정 내용:

  1. 개행으로 메시지가 전송되는 버그 수정
    메시지를 입력할 때 개행을 사용하더라도 전송 버튼을 눌러야만 개행이 반영되는 것이 정상입니다. 그러나 어떤 이유로 개행이 그대로 전송되는 버그가 발생했습니다. 이 버그를 수정함으로써, 개행이 텍스트 영역 내에서 정상적으로 반영되고, 메시지가 전송될 때 개행 없이 전송됩니다.

  2. 더 보기 버튼이 작동하지 않는 문제 수정
    '더 보기' 버튼이 기능하지 않던 문제도 수정되었습니다. 이 버튼은 클릭하면 더 많은 콘텐츠가 표시되는 기능인데, 어떤 이유로 이 기능이 중단되었었습니다. 이 문제를 해결함으로써, '더 보기' 버튼이 정상적으로 작동하여 사용자가 더 많은 콘텐츠를 볼 수 있게 됩니다.

Why

Additional info (optional)

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Update CHANGELOG_CHERRYPICK.md
  • (If possible) Add tests

@noridev
Copy link
Collaborator

noridev commented Feb 14, 2025

この問題は、長年にわたって私たちを悩ませてきました。
問題を解決しようとテストをする過程で、Misskeyのチャットがなくなった時点でもこの問題があることを確認しました。
(それだけMisskey側がチャットに関心がなかったのでしょう...?)

最近チャットを使うユーザーが増えたことで、この問題が浮上して解決に乗り出したが、適当な解決方法を見つけられずにいましたが、PRしてくださったコードを見ると、本当に簡単に解決できる問題でしたね。🥲

既存のMkPaginationの動作に影響を及ぼさないようにコードを一部修正しました。
修正されたコードはローカルテスト環境で同じ動作を確認しており、MkPaginationコンポーネントを使用する他のページでも問題なく動作すると思います。

この場を借りて、CherryPickを使用するすべてのユーザーを代表して、CherryPickの長年の問題を解決してくださって本当にありがとうございます。🙏🙏

@noridev noridev merged commit 555ad29 into kokonect-link:develop Feb 14, 2025
1 of 2 checks passed
noridev added a commit that referenced this pull request Feb 14, 2025
  - `더 보기`를 눌러 이전 대화를 볼 수 없음
  - 개행만으로 메시지를 보낼 수 있음
    - 이 변경으로 더 이상 첫 줄에 개행을 입력할 수 없게 됩니다.
@noridev noridev mentioned this pull request Feb 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants