-
-
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
新規にフォローした人のwithRepliesをtrueにする機能を追加 #12048
新規にフォローした人のwithRepliesをtrueにする機能を追加 #12048
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #12048 +/- ##
===========================================
+ Coverage 79.26% 79.39% +0.12%
===========================================
Files 930 931 +1
Lines 99219 99364 +145
Branches 8065 8088 +23
===========================================
+ Hits 78646 78889 +243
+ Misses 20573 20475 -98
☔ View full report in Codecov by Sentry. |
|
This comment was marked as resolved.
This comment was marked as resolved.
あとはフォローインポートでサーバーに情報ほしいと思う |
where: 'account' であればサーバーに保存される |
ふーむ |
なるほどです。これは勘違いしてました。 |
db92f2c
to
8054ce9
Compare
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.
クライアント設定の中にあるのは違うかも(他に良い場所は思いつかないけど)
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.
若干違和感はありますが元々のshowPreviewを一応参考にしました
+ フォローインポート時にwithReplies: trueとしてインポートするかどうか設定可能にする というやり方もありそうと思った |
たしかにそっちに選択があってもいいですね。その方針にしてみます |
importは試せてないですが普通のフォローボタンは動作確認までしました |
importもどうさかくにんしました。 |
frontendのlintの問題が直ればあとはYOSASOU |
lint、どう処理するのが正しいですかね |
lintは通るようになりましたがe2eが落ちてる... |
e2eが落ちてるのは関係なさそう |
👍🏻 👍🏻 👍🏻 |
What
新規にフォローした人のwithRepliesをtrueにする機能を追加する
Why
解決したい問題は #12009 や #12006 と同様にすべてのユーザについてwithRepliesをオンにするのが著しく面倒なことで、このPullReqではこの問題を新規にフォローする人について解決します。
Additional info (optional)
Checklist