Skip to content

enhance(frontend): チャンネルへのノート投稿導線を改善#13028

Draft
mesichicken wants to merge 60 commits intomisskey-dev:developfrom
mesichicken:feature/default-post-target-detect-from-path
Draft

enhance(frontend): チャンネルへのノート投稿導線を改善#13028
mesichicken wants to merge 60 commits intomisskey-dev:developfrom
mesichicken:feature/default-post-target-detect-from-path

Conversation

@mesichicken
Copy link
Copy Markdown

@mesichicken mesichicken commented Jan 18, 2024

Fix #10762
Fix #12711
Fix #12712
Fix #14549

What

開いているページ(mainRouter)がチャンネルのとき、そのチャンネルを投稿先にしたいです

290973888-edc2f8b0-270d-411c-891c-e46a2f92afc5.mov

Why

投稿ボタンが2つあるため間違えが発生しやすく、NとPのショートカットでチャンネルへ投稿できないため

Additional info (optional)

#12688
こちらのPRを再度OPENします。

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

@github-actions github-actions Bot added the packages/frontend Client side specific issue/PR label Jan 18, 2024
@kakkokari-gtyih
Copy link
Copy Markdown
Contributor

kakkokari-gtyih commented Jan 18, 2024

@mesichicken 元のやつがOpenになった

@fruitriin
Copy link
Copy Markdown
Contributor

おっふ

@mesichicken
Copy link
Copy Markdown
Author

あら

@fruitriin
Copy link
Copy Markdown
Contributor

なおせちゃったら直しちゃってください!

@mesichicken
Copy link
Copy Markdown
Author

@kakkokari-gtyih
私のほうで直せるか試してみます。無理そうでしたらcloseします。

@kakkokari-gtyih
Copy link
Copy Markdown
Contributor

元のほうが閉じられたのでこっちベースで進めましょうか

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 18, 2024

Codecov Report

❌ Patch coverage is 0% with 26 lines in your changes missing coverage. Please review.
✅ Project coverage is 13.84%. Comparing base (00cbd1e) to head (7344325).
⚠️ Report is 72 commits behind head on develop.

Files with missing lines Patch % Lines
packages/frontend/src/components/MkPostForm.vue 0.00% 18 Missing and 8 partials ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #13028       +/-   ##
============================================
- Coverage    63.49%   13.84%   -49.66%     
============================================
  Files         1161      241      -920     
  Lines       115939    11664   -104275     
  Branches      8351     3953     -4398     
============================================
- Hits         73619     1615    -72004     
+ Misses       40114     7882    -32232     
+ Partials      2206     2167       -39     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mesichicken mesichicken marked this pull request as ready for review January 18, 2024 07:06
@mesichicken mesichicken changed the title [WIP]開いてるページで投稿先がチャンネルになってほしい 開いてるページで投稿先がチャンネルになってほしい Jan 18, 2024
@syuilo
Copy link
Copy Markdown
Member

syuilo commented Jan 19, 2024

これだとチャンネルを開いている時にチャンネル外に投稿する手段がなくなりそう?

@Sayamame-beans
Copy link
Copy Markdown
Member

Sayamame-beans commented Jan 19, 2024

これだとチャンネルを開いている時にチャンネル外に投稿する手段がなくなりそう?

デッキUIでないケースでは、それで良いのではないかと思います…?
(標準でチャンネル外に投稿出来てしまう挙動で困っていたみたいなので)

一つの選択肢としては、#12687 も加えて、非デッキUIでのチャンネル表示時はデフォルトの公開範囲がそのチャンネルになるが、公開範囲を変えられるのでチャンネル外にも投稿出来るようにする、とかでしょうか…?

@syuilo
Copy link
Copy Markdown
Member

syuilo commented Jan 19, 2024

デッキUIでないケースでは、それで良いのではないかと思います…?
(標準でチャンネル外に投稿出来てしまう挙動で困っていたみたいなので)

チャンネル開きながらチャンネル外に投稿するシチュエーションは普通にあるわね

@samunohito samunohito removed this from the v2024.11.1? milestone Nov 23, 2024
@kakkokari-gtyih kakkokari-gtyih modified the milestones: v2025.2.0, v2025.2.1 Feb 5, 2025
@kakkokari-gtyih
Copy link
Copy Markdown
Contributor

コンフリクト解消

@kakkokari-gtyih
Copy link
Copy Markdown
Contributor

コンフリクト解消

テストがあるとよさそうだわね (結構複雑に見えるので、テストなしに今後メンテナンスしていくのは難しそう)

テストを書ける人を募集しています

@kakkokari-gtyih kakkokari-gtyih modified the milestones: v2025.2.1, [DO NOT MERGE THESE YET] v2025.3.0~, [DO NOT MERGE THESE YET] v2025.3.1~, v2025.3.0 Feb 24, 2025
@samunohito samunohito modified the milestones: v2025.3.3, v2025.5.x Apr 15, 2025
@Sayamame-beans Sayamame-beans moved this from Todo to Stale in [実験中] 管理用 Aug 11, 2025
@dosubot dosubot Bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Oct 9, 2025
@kakkokari-gtyih
Copy link
Copy Markdown
Contributor

コンフリクト解消

@fruitriin
Copy link
Copy Markdown
Contributor

fruitriin commented Feb 2, 2026

この機能を取り込んだサーバーの住人にはとても好評なので、なにかコメントがつくことを祈っています

@kakkokari-gtyih kakkokari-gtyih marked this pull request as draft February 2, 2026 16:00
@kakkokari-gtyih
Copy link
Copy Markdown
Contributor

kakkokari-gtyih commented Feb 2, 2026

と思ったけどしばらく実動作確認挟んでない(コード上でのコンフリクト解消のみ)のでしてからのほうが良いわね

@fruitriin
Copy link
Copy Markdown
Contributor

この機能を取り込んだサーバーの住人にはとても好評なので、なにかコメントがつくことを祈っています

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment