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

通知のフィルタができない #10550

Closed
tamaina opened this issue Apr 9, 2023 · 9 comments
Closed

通知のフィルタができない #10550

tamaina opened this issue Apr 9, 2023 · 9 comments
Labels
🐛Bug Unexpected behavior

Comments

@tamaina
Copy link
Contributor

tamaina commented Apr 9, 2023

通知はありませんになる

image

@tamaina tamaina added ⚠️bug? This might be a bug 🐛Bug Unexpected behavior and removed ⚠️bug? This might be a bug labels Apr 9, 2023
@tamaina
Copy link
Contributor Author

tamaina commented Apr 9, 2023

なぜかリアクションだけはできる?

@tamaina
Copy link
Contributor Author

tamaina commented Apr 9, 2023

謎というか条件がわからない

@tamaina
Copy link
Contributor Author

tamaina commented Apr 9, 2023

(ログとか通信とかをまだ調べてない

@mappi-pr
Copy link
Contributor

@tamaina
フィルタしたときに、(まず1回)Limit10件でAPIリクエストしていたので
画面の表示上にフィルタ対象となる項目が存在しても、
配列の11件目以降になっているとフィルタに拾われずに「通知はありません」が表示されてしまうようです。

■フィルタ成功パターン
・フィルタ対象が10件目以内にある場合

・フィルタが成功する

■フィルタ失敗パターン
・フィルタ対象が11件目以降にある場合

・フィルタが失敗する

【参考情報】
10件目以内にフィルタ対象が存在すると、Limit30件で再度リクエストを投げて取得できた情報でフィルタしてました。

@mappi-pr
Copy link
Contributor

・フィルタしたときのAPIリクエスト

@mappi-pr
Copy link
Contributor

(もしかしたら)フィルタかけるたびにpaginationされている?
さっきioでフィルタ連打していたらRatelimitに引っかかったような挙動が見えたので

const pagination: Paging = {
endpoint: 'i/notifications' as const,
limit: 10,
params: computed(() => ({
includeTypes: props.includeTypes ?? undefined,
excludeTypes: props.includeTypes ? undefined : $i.mutingNotificationTypes,
})),
};

@syuilo
Copy link
Member

syuilo commented May 26, 2023

される

@Sayamame-beans
Copy link
Member

自動じゃなくて良いので"もっと見る"(もっと読み込む?)ボタンが表示されると良さそうですね…(2024.2.0の通知カラム(デッキ)と通知ページのフィルタで確認)

@tamaina
Copy link
Contributor Author

tamaina commented Feb 20, 2024

Duplicate of #13413

@tamaina tamaina marked this as a duplicate of #13413 Feb 20, 2024
@tamaina tamaina closed this as not planned Won't fix, can't repro, duplicate, stale Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛Bug Unexpected behavior
Projects
None yet
Development

No branches or pull requests

4 participants