-
Notifications
You must be signed in to change notification settings - Fork 23
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
Miria v2 #572
Comments
require_trailing_commas を入れてもらいたいです(インデントが揃うし差分が見やすくなる) |
|
負債が多すぎてなきそう |
AriaではAPIの呼び出しは基本的に全て Notifierのbuildの中でasyncを書くとデータとローディングとエラーの3状態のsealed classみたいなやつを勝手に返してくれるので、switchでそれぞれのviewをまとめて書けます |
riverpodの2585でもにょってる ん~~~もうriverpod 3.0-dev.3にするか |
riverpod_generatorに移行するならriverpod_lintを入れてみてもいいかもしれません |
riverpodのドキュメントを読み直してたらflutter_hooks使うと便利なのでは?ってなってきた |
ぬぁーーー ChangeNotifierはriverpod_generatorでlistenされない そういう仕様らしい |
Miriaをごっそりリファクタリングする
unawaited_futures
discarded_futures
とかを入れるshowDialog
を使わずに、ダイアログの状態管理をするStateNotifier
を作ってその状態を受けて表示できるようにする (Misskey本家も割とそんな感じの動きしてる)async gap
の対策StateNotifier
のFuture
とかで一連の処理でユーザーのダイアログ操作待ちをさせたいときがある 画像つきノート投稿時のサーバーからのセンシティブ判定とかshowDialog
やshowModalBottomSheet
を auto_route で呼び出すようにするThe text was updated successfully, but these errors were encountered: