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: media silence #13842

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

Conversation

anatawa12
Copy link
Member

@anatawa12 anatawa12 commented May 20, 2024

What

Fixes #13621

絵文字も実装

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

@github-actions github-actions bot added packages/frontend Client side specific issue/PR packages/backend Server side specific issue/PR packages/misskey-js labels May 20, 2024
Copy link

codecov bot commented May 20, 2024

Codecov Report

Attention: Patch coverage is 65.45455% with 19 lines in your changes are missing coverage. Please review.

Project coverage is 66.41%. Comparing base (e0b4799) to head (217a3f6).

Files Patch % Lines
...kend/src/server/api/endpoints/admin/update-meta.ts 46.66% 8 Missing ⚠️
packages/backend/src/core/ReactionService.ts 0.00% 7 Missing ⚠️
packages/backend/src/core/UtilityService.ts 66.66% 2 Missing ⚠️
...backend/src/core/entities/InstanceEntityService.ts 0.00% 1 Missing ⚠️
...ges/backend/src/server/api/endpoints/admin/meta.ts 90.90% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #13842      +/-   ##
===========================================
- Coverage    66.42%   66.41%   -0.02%     
===========================================
  Files          990      989       -1     
  Lines       116887   116876      -11     
  Branches      4503     5829    +1326     
===========================================
- Hits         77647    77619      -28     
+ Misses       39209    37817    -1392     
- Partials        31     1440    +1409     

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

Copy link
Contributor

github-actions bot commented May 20, 2024

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

差分はこちら
--- base
+++ head
@@ -163,6 +163,12 @@
                         "type": "string"
                       }
                     },
+                    "mediaSilencedHosts": {
+                      "type": "array",
+                      "items": {
+                        "type": "string"
+                      }
+                    },
                     "pinnedUsers": {
                       "type": "array",
                       "items": {
@@ -573,6 +579,7 @@
                     "enableEmail",
                     "enableServiceWorker",
                     "translatorAvailable",
+                    "mediaSilencedHosts",
                     "pinnedUsers",
                     "hiddenTags",
                     "blockedHosts",
@@ -12950,6 +12957,15 @@
                       "type": "string"
                     }
                   },
+                  "mediaSilencedHosts": {
+                    "type": [
+                      "array",
+                      "null"
+                    ],
+                    "items": {
+                      "type": "string"
+                    }
+                  },
                   "summalyProxy": {
                     "type": [
                       "string",
@@ -77723,6 +77739,9 @@
           "isSilenced": {
             "type": "boolean"
           },
+          "isMediaSilenced": {
+            "type": "boolean"
+          },
           "iconUrl": {
             "type": [
               "string",
@@ -77784,6 +77803,7 @@
           "maintainerName",
           "maintainerEmail",
           "isSilenced",
+          "isMediaSilenced",
           "iconUrl",
           "faviconUrl",
           "themeColor",

Get diff files from Workflow Page

@anatawa12 anatawa12 marked this pull request as ready for review May 21, 2024 03:54
locales/ja-JP.yml Outdated Show resolved Hide resolved
@anatawa12
Copy link
Member Author

コンフリクト解消

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/backend Server side specific issue/PR packages/frontend Client side specific issue/PR packages/misskey-js
Projects
None yet
Development

Successfully merging this pull request may close these issues.

あるサーバーから流れてくるメディアをすべて閲覧注意に出来るようにしたい
2 participants