feat: ユーザのプロフィールとノートの名前欄横にロールバッジを出す機能を追加 #1937
Conversation
| @@ -45,6 +45,7 @@ object DbModule { | |||
| .addMigrations(MIGRATION_8_10) | |||
| .addMigrations(MIGRATION_10_11) | |||
| .addMigrations(MIGRATION_51_52) | |||
| .addMigrations(MIGRATION_56_57) | |||
There was a problem hiding this comment.
AutoMigrationでMigrationしているので手動でMigrationをする必要性はないと思っているのですが、
何かMIGRATIONを実行しないといけない理由などはある感じでしょうか？
There was a problem hiding this comment.
AutoMigrationではうまく対応できない場合はAutoMigrationの方は削除しても良いと思いました。
反対にAutoMigrationで問題ない場合は手動でMigrationのコードを書く必要性はないと考えているのですが他に理由がある場合ご教授いただけると助かります。
There was a problem hiding this comment.
なるほど…こちら、調査が足りなかったようです。
どちらかでうまくマイグレーションできないか試してみて、正常にいけたら再度プルリクさせてください:pray:
| app:layout_constraintStart_toEndOf="@id/mainName" | ||
| app:layout_constraintTop_toTopOf="parent" | ||
| tools:text="ユーザー名awefawefawefawefawefawefwaefwef" /> | ||
|
|
||
| <androidx.compose.ui.platform.ComposeView |
There was a problem hiding this comment.
RecyclerViewとComposeの組み合わせはあまりパフォーマンス的によくないのではないか？と思っているのですが、ただ今回のRoleの仕様上、表示する件数が決まっていないため、
Android Viewで実装した場合はLinearLayoutやConstraintLayoutのFlexに
コードで動的にImageViewなどをinflateしていく必要性があり、
ComposeViewの方がパフォーマンスがいいようにも思えます。
なので自分としてはパフォーマンス的にどちらが良いのかあまりわかっていません。
このことについて何か知見があったりしますでしょうか？
お手数でしたらマージ後に私の方で検証したいと考えています。
There was a problem hiding this comment.
RecyclerViewとComposeの組み合わせはあまりパフォーマンス的によくない
はずかしながら、初めて知りました…
記事を見る限り、明らかにパフォーマンスが劣化しているようです。
Composeはやめて、RecyclerViewを使う実装に寄せたほうが良いでしょうか…？
|
単体テストはこちらで落ちているようです |
|
今回のPRでは十分だと思ったので一旦マージしたいと思います！ |
やったこと
ユーザのプロフィール画面とノートに対し、ユーザに割り当てられたロールのアイコンを出すようにViewを追加しました。
ただし、リプライ時のリプライ先ノートと引用リノートの引用された側ノートは対象外としています。時間表示も省かれていることから、画面表示の見通しのためにあえて削っているのかなと推測しました。
動作確認
端末上で表示確認
スクリーンショット(任意)
備考
すでにユーザ情報をDBに持っているため、それにあわせてロールの情報も持つようにDB定義を更新しています。
この機能リクエストが不要でしたら、お手数ですがcloseお願い致します。
Issue番号