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

コメント数制限をv0.24.3にも追加する #319

Closed
takahashim opened this issue Dec 26, 2021 · 1 comment
Closed

コメント数制限をv0.24.3にも追加する #319

takahashim opened this issue Dec 26, 2021 · 1 comment
Labels
bug Something isn't working

Comments

@takahashim
Copy link
Collaborator

対象ドメイン/ The domain

  • すべて

起こっている問題 / The Problem

#285 のv0.24.3へのアップデートについては #312 にて作業中ですが、 #285 にも書いたとおり、decidim-comments周りが大幅に変更されたため、 #278 の対応を改めて行う必要があります。

Reactじゃなくのである意味簡単かな? と思いきやjQueryとFoundationで何とかするロジックが入っていて細かい挙動の制御が厳しい…。

期待する見せ方・挙動 / Expected Behavior

  • デフォルトではコメントが大量に表示されない
    • 上限は環境変数で設定できる
  • 上限が超えた場合には「すべてのコメントを表示する」ボタンが出る
  • 上記ボタンをクリックすると、コメントがすべて表示される

起こっている問題の再現手段 / Steps to Reproduce

  1. アップデートした環境で、大量にコメントを書く
  2. 提案などで、コメントページを表示する

動作環境・ブラウザ / Environment

  • すべて
@takahashim takahashim added the bug Something isn't working label Dec 26, 2021
@takahashim
Copy link
Collaborator Author

#320 で対応しましたが、どうもReactが廃止されてもXHRで定期的にコメントの参照と更新をJS(jQuery)で行っているようで、そっちの方はあんまりちゃんと対応できてないです。
というかどう表示させればいいのか…。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants