Repository navigation
fix(offline): オフラインモードの検査を通らない経路と、切替の途中の抜けを塞ぐ (#1350) - #1396
Merged
Merged
Conversation
リリース前のレビューの指摘を反映する。 - アプリの状態確認 (自己診断) が、検査なしのクライアントで Misskey に 問い合わせていた。門を呼び出し側が渡す形にし、アプリはオフラインの 門を、コマンドラインは通す門を渡す - 全アカウントのカラム: オフライン中にキャッシュの末尾まで読むと 「尽きた」と記録され、解除後もそれより古いノートを読み込めなかった - 投稿 / ログアウト / データの削除: 確認ダイアログを待つ間にオフラインに 入った場合も止める (投稿は、閉じてから送ると本文を失う) - 巡回: 途中でオフラインに入ったら、AI を呼ぶ直前で見送る - 接続: ロックを待つ間に門が閉じたら張らない - 凍結の照会: 途中で止められたら、残りを戻して再開で流す - チャットカラム: 取得の途中でオフラインに入っても、解除で取り直す - 起動時の読み込みに失敗した後で切替に成功したら、購読を張り直す Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 41 minutes. View limit details
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. |
github-actions
Bot
added
the
javascript
Deploying notedeck with
|
| Latest commit: |
e099fba
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://2e9e0e93.notedeck-d3a.pages.dev |
| Branch Preview URL: | https://fix-offline-review-followups.notedeck-d3a.pages.dev |
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
何を直すか
Release PR (#1395) に付いたレビューの指摘 10 件のうち 9 件。リリースに入れる前に直す。
コードで確かめて、実際の不具合だったもの
doctor::diagnoseが検査なしのクライアントを固定で作っていて、アプリの状態確認もこれを呼ぶ。手動のオフライン中に開くと Misskey に要求が出る。門を引数で受ける形にし、アプリはオフラインの門を、コマンドラインは通す門を渡す切替の途中の抜け (起きにくいが、安く塞げるもの)
入れていないもの (1 件)
「オフラインに入ると照会の購読の対応が残り、解除後にライブ更新が再開されない可能性」。オフラインに入るときに接続を切る動きは今回の変更より前から同じで、今回の変更で起きるようになったものではない。解除後に元に戻ることは実機で確認されている。ライブ更新が再開されるかを個別に確かめる必要があるので、別に扱う。
確認
Refs #1350
🤖 Generated with Claude Code