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

feat: アンテナでセンシティブなチャンネルからのノートを除外できるように #15346

Open
wants to merge 27 commits into
base: develop
Choose a base branch
from

Conversation

sevenc-nanashi
Copy link
Contributor

@sevenc-nanashi sevenc-nanashi commented Jan 25, 2025

What

タイトル通りです。

Why

closes: #14177

Additional info (optional)

手元の環境ではなぜかESLintのフォーマットが効かなかったので手動フォーマットしています。(おそらく何か間違えている...)

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 packages/frontend Client side specific issue/PR packages/backend Server side specific issue/PR packages/misskey-js packages/backend:test labels Jan 25, 2025
Copy link

codecov bot commented Jan 25, 2025

Codecov Report

Attention: Patch coverage is 57.14286% with 12 lines in your changes missing coverage. Please review.

Project coverage is 43.88%. Comparing base (2918fb2) to head (c743a3a).

Files with missing lines Patch % Lines
...ckages/frontend/src/components/MkAntennaEditor.vue 0.00% 4 Missing ⚠️
...backend/src/server/api/endpoints/antennas/notes.ts 0.00% 3 Missing ⚠️
packages/backend/src/core/AntennaService.ts 0.00% 2 Missing ⚠️
.../backend/src/core/entities/AntennaEntityService.ts 0.00% 1 Missing ⚠️
...ackend/src/server/api/endpoints/antennas/create.ts 50.00% 1 Missing ⚠️
...ackend/src/server/api/endpoints/antennas/update.ts 50.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #15346       +/-   ##
============================================
+ Coverage    15.34%   43.88%   +28.54%     
============================================
  Files          785     1627      +842     
  Lines        72098   171634    +99536     
  Branches      1279     4143     +2864     
============================================
+ Hits         11066    75328    +64262     
- Misses       60608    95829    +35221     
- Partials       424      477       +53     

☔ 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.

Copy link
Contributor

github-actions bot commented Jan 25, 2025

このPRによるapi.jsonの差分

差分はこちら
--- base
+++ head
@@ -18560,6 +18560,9 @@
                   },
                   "withFile": {
                     "type": "boolean"
+                  },
+                  "hideNotesInSensitiveChannel": {
+                    "type": "boolean"
                   }
                 },
                 "required": [
@@ -19461,6 +19464,9 @@
                   },
                   "withFile": {
                     "type": "boolean"
+                  },
+                  "hideNotesInSensitiveChannel": {
+                    "type": "boolean"
                   }
                 },
                 "required": [
@@ -81884,6 +81890,10 @@
           "notify": {
             "type": "boolean",
             "default": false
+          },
+          "hideNotesInSensitiveChannel": {
+            "type": "boolean",
+            "default": false
           }
         },
         "required": [
@@ -81902,7 +81912,8 @@
           "withFile",
           "isActive",
           "hasUnreadNote",
-          "notify"
+          "notify",
+          "hideNotesInSensitiveChannel"
         ]
       },
       "Clip": {

Get diff files from Workflow Page

@@ -626,6 +629,42 @@ describe('アンテナ', () => {
assert.deepStrictEqual(response, expected);
});

test('が取得できること(センシティブチャンネルのノートを除く)', async () => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

これだけチャンネル作成が必要になってくるので上のeachとは別に書いています。

@kakkokari-gtyih kakkokari-gtyih added this to the [DO NOT MERGE THESE YET] v2025.2.0~ milestone Jan 26, 2025
@sevenc-nanashi sevenc-nanashi force-pushed the feat/hide-sensitive-from-antenna branch from d8c380a to f98076e Compare January 26, 2025 10:08
@kakkokari-gtyih kakkokari-gtyih modified the milestones: [DO NOT MERGE THESE YET] v2025.3.0~, v2025.2.0 Jan 28, 2025
@sevenc-nanashi
Copy link
Contributor Author

sevenc-nanashi commented Feb 20, 2025

そういえば:このPRってどういう状況なんでしょうか?(次のMisskey Office Hour待ちみたいな感じとか?)

@samunohito samunohito added this to the [DO NOT MERGE THESE YET] v2025.3.0~ milestone Feb 20, 2025
@samunohito
Copy link
Member

明日レビューします!

Copy link
Member

@samunohito samunohito left a comment

Choose a reason for hiding this comment

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

一通り見ましたが、明日の朝にもう一回見たいので見ます

Copy link
Member

@samunohito samunohito left a comment

Choose a reason for hiding this comment

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

🙏

@sevenc-nanashi
Copy link
Contributor Author

CHANGELOGをUnreleasedのところに移動しました。よろしくお願いします。

@kakkokari-gtyih kakkokari-gtyih removed this from the [DO NOT MERGE THESE YET] milestone Mar 3, 2025
@sevenc-nanashi
Copy link
Contributor Author

(このPRってどういう状態なんでしょう?)

@samunohito samunohito added this to the v2025.3.1 milestone Mar 6, 2025
@samunohito
Copy link
Member

他に緊急度の高い問題があり、優先度の兼ね合いからマージ待ちのままになっています。

@kakkokari-gtyih kakkokari-gtyih modified the milestones: v2025.3.1, v2025.3.2 Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/backend:test packages/backend Server side specific issue/PR packages/frontend Client side specific issue/PR packages/misskey-js
Projects
Status: Approved
Development

Successfully merging this pull request may close these issues.

センシティブチャンネルをアンテナから除外できる設定をつけてほしい
5 participants