-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
enhance(frontend): アンテナ、リスト等の名前をdeckのカラム名のデフォルト値にするように #13992
enhance(frontend): アンテナ、リスト等の名前をdeckのカラム名のデフォルト値にするように #13992
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #13992 +/- ##
========================================
Coverage 18.81% 18.82%
========================================
Files 766 766
Lines 111341 111400 +59
Branches 1234 1234
========================================
+ Hits 20951 20966 +15
- Misses 89801 89845 +44
Partials 589 589 ☔ View full report in Codecov by Sentry. |
lintがあかんかも |
dockleに気を取られて気づいてなかったですありがとうございます |
直しました(fix on saveが切れてた) |
Conflict解消 |
既に名前が指定されている場合でもデフォルト値用の名前取得のためだけにリクエストが発火されるのが無駄かもしれない(もしくはわたしが何か見逃してる…?) |
たしかにそうですね。nameがnull or emptyのときのみにしたほうが良いですね |
不要な場合には取得しないようにしました。 |
commit a28ab48 Author: anatawa12 <anatawa12@icloud.com> Date: Thu Jan 9 11:02:25 2025 +0900 docs: fix changelog commit 212a2d8 Author: anatawa12 <anatawa12@icloud.com> Date: Thu Jan 9 10:50:26 2025 +0900 chore: チャンネルについては投稿でも使用されてる channel 変数を使用するように commit e173d2b Author: anatawa12 <anatawa12@icloud.com> Date: Thu Jan 9 10:48:54 2025 +0900 chore: カラム名が指定されている場合にはチャンネル名を取得しないように commit 6209947 Merge: c4c3c86 13439e0 Author: かっこかり <67428053+kakkokari-gtyih@users.noreply.github.com> Date: Wed Jan 8 21:13:54 2025 +0900 Merge branch 'develop' into deck-default-name commit c4c3c86 Merge: cda207f 89841e4 Author: かっこかり <67428053+kakkokari-gtyih@users.noreply.github.com> Date: Thu Sep 26 13:31:29 2024 +0900 Merge branch 'develop' into deck-default-name commit cda207f Merge: a805a27 8b4933c Author: Sayamame-beans <61457993+Sayamame-beans@users.noreply.github.com> Date: Wed Jul 17 19:39:59 2024 +0900 Merge branch 'develop' into deck-default-name commit a805a27 Merge: eeb1d73 b8b4dc5 Author: syuilo <4439005+syuilo@users.noreply.github.com> Date: Sat Jun 22 19:45:39 2024 +0900 Merge branch 'develop' into deck-default-name commit eeb1d73 Author: anatawa12 <anatawa12@icloud.com> Date: Fri Jun 14 15:27:16 2024 +0900 lint: fix commit 063cd34 Author: anatawa12 <anatawa12@icloud.com> Date: Fri Jun 14 12:13:35 2024 +0900 docs: アンテナ、リスト等の名前をカラム名のデフォルト値にするように commit 90efe12 Author: anatawa12 <anatawa12@icloud.com> Date: Fri Jun 14 12:10:33 2024 +0900 feat: アンテナ、リスト等の名前をカラム名のデフォルト値にするように commit 452a8ed Author: anatawa12 <anatawa12@icloud.com> Date: Fri Jun 14 10:42:15 2024 +0900 refactor: remove type errors from deck.vue and deck-store.ts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
コンフリクト解消だけしておいいてください
607bf60
into
misskey-dev:develop
Thank you 🙏 |
…3992) * refactor: remove type errors from deck.vue and deck-store.ts * feat: アンテナ、リスト等の名前をカラム名のデフォルト値にするように * docs: アンテナ、リスト等の名前をカラム名のデフォルト値にするように * lint: fix * chore: カラム名が指定されている場合にはチャンネル名を取得しないように * chore: チャンネルについては投稿でも使用されてる channel 変数を使用するように * docs: fix changelog --------- Co-authored-by: syuilo <4439005+syuilo@users.noreply.github.com> Co-authored-by: Sayamame-beans <61457993+Sayamame-beans@users.noreply.github.com> Co-authored-by: かっこかり <67428053+kakkokari-gtyih@users.noreply.github.com>
…3992) * refactor: remove type errors from deck.vue and deck-store.ts * feat: アンテナ、リスト等の名前をカラム名のデフォルト値にするように * docs: アンテナ、リスト等の名前をカラム名のデフォルト値にするように * lint: fix * chore: カラム名が指定されている場合にはチャンネル名を取得しないように * chore: チャンネルについては投稿でも使用されてる channel 変数を使用するように * docs: fix changelog --------- Co-authored-by: syuilo <4439005+syuilo@users.noreply.github.com> Co-authored-by: Sayamame-beans <61457993+Sayamame-beans@users.noreply.github.com> Co-authored-by: かっこかり <67428053+kakkokari-gtyih@users.noreply.github.com>
What
Fixes #11077
Why
Additional info (optional)
Checklist