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

enhance(frontend): アンテナ、リスト等の名前をdeckのカラム名のデフォルト値にするように #13992

Merged
merged 13 commits into from
Feb 7, 2025

Conversation

anatawa12
Copy link
Member

What

Fixes #11077

Why

Additional info (optional)

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@anatawa12 anatawa12 changed the title アンテナ、リスト等の名前をカラム名のデフォルト値にするように アンテナ、リスト等の名前をdeckのカラム名のデフォルト値にするように Jun 14, 2024
@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Jun 14, 2024
Copy link

codecov bot commented Jun 14, 2024

Codecov Report

Attention: Patch coverage is 17.85714% with 69 lines in your changes missing coverage. Please review.

Project coverage is 18.82%. Comparing base (d008394) to head (67f8fed).
Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
packages/frontend/src/ui/deck/list-column.vue 0.00% 15 Missing ⚠️
packages/frontend/src/ui/deck/channel-column.vue 0.00% 14 Missing ⚠️
packages/frontend/src/ui/deck/deck-store.ts 0.00% 10 Missing ⚠️
...ages/frontend/src/ui/deck/role-timeline-column.vue 0.00% 10 Missing ⚠️
packages/frontend/src/ui/deck/antenna-column.vue 0.00% 9 Missing ⚠️
packages/frontend/src/ui/deck/column.vue 0.00% 3 Missing ⚠️
packages/frontend/src/ui/deck/direct-column.vue 0.00% 2 Missing ⚠️
packages/frontend/src/ui/deck/mentions-column.vue 0.00% 2 Missing ⚠️
packages/frontend/src/ui/deck.vue 0.00% 1 Missing ⚠️
...ages/frontend/src/ui/deck/notifications-column.vue 0.00% 1 Missing ⚠️
... and 2 more
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.
📢 Have feedback on the report? Share it here.

@kakkokari-gtyih
Copy link
Contributor

lintがあかんかも

@anatawa12
Copy link
Member Author

dockleに気を取られて気づいてなかったですありがとうございます

@anatawa12
Copy link
Member Author

直しました(fix on saveが切れてた)

@Sayamame-beans
Copy link
Member

Conflict解消

@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Jan 8, 2025

既に名前が指定されている場合でもデフォルト値用の名前取得のためだけにリクエストが発火されるのが無駄かもしれない(もしくはわたしが何か見逃してる…?)

@anatawa12
Copy link
Member Author

既に名前が指定されている場合でもデフォルト値用の名前取得のためだけにリクエストが発火されるのが無駄かもしれない(もしくはわたしが何か見逃してる…?)

たしかにそうですね。nameがnull or emptyのときのみにしたほうが良いですね

@anatawa12
Copy link
Member Author

不要な場合には取得しないようにしました。

anatawa12 added a commit to anatawa12/misskey that referenced this pull request Jan 28, 2025
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
@kakkokari-gtyih kakkokari-gtyih added this to the v2025.2.1 milestone Feb 5, 2025
Copy link
Contributor

@kakkokari-gtyih kakkokari-gtyih left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コンフリクト解消だけしておいいてください

@kakkokari-gtyih kakkokari-gtyih changed the title アンテナ、リスト等の名前をdeckのカラム名のデフォルト値にするように enhance(frontend): アンテナ、リスト等の名前をdeckのカラム名のデフォルト値にするように Feb 5, 2025
@mi-gh-maintainer mi-gh-maintainer bot merged commit 607bf60 into misskey-dev:develop Feb 7, 2025
8 of 9 checks passed
Copy link

Thank you 🙏

@anatawa12 anatawa12 deleted the deck-default-name branch February 7, 2025 08:09
DA-TENSHI pushed a commit to SHINANOSKEY-Projekt/SHINANOSKEY that referenced this pull request Feb 7, 2025
…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>
LemonDouble pushed a commit to LemonDouble/misskey that referenced this pull request Mar 1, 2025
…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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
Development

Successfully merging this pull request may close these issues.

Deckのカラム追加時にアンテナ、リスト等の名前をカラムの名前にしてほしい
4 participants