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.

Sign up for GitHub

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

Jump to bottom

fix(backend): アンテナ等がポリシーで定められた上限を超えている場合、変更や追加ができないように #646

Merged
merged 5 commits into from
Jun 16, 2024
Merged

fix(backend): アンテナ等がポリシーで定められた上限を超えている場合、変更や追加ができないように #646

merged 5 commits into from
Jun 16, 2024

Conversation

kabo2468
Copy link

@kabo2468 kabo2468 commented May 30, 2024 •
edited
Loading

What

  • ポリシーで制限されている数 + 1まで作成、追加できるのを修正
  • アンテナ、Webhook、リスト、クリップの数の上限を超えている場合に、編集や変更、ユーザーの追加をできないように

Why

Additional info (optional)

とりあえずe2eテストのコードも直したけど何故か+1でも+5でも通るので謎

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

@kabo2468
Copy link
Author

ローカルでテスト済み

下2つができるけどできないようにするかどうか

  • クリップの上限は超えてないけどクリップのノート数が超えているときに、クリップの編集ができる
  • リストの上限は超えてないけどリストのユーザー数が超えているときに、リストの編集ができる

@kakkokari-gtyih
Copy link

上限を超えている場合でも減らすことはできたほうが良さそう？

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
4 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@u1-liquid u1-liquid merged commit 2b2975c into MisskeyIO:io Jun 16, 2024
14 of 15 checks passed
Astro-CQM pushed a commit to hotomoe/hotomoe that referenced this pull request Jun 19, 2024
@kabo2468 @Astro-CQM
fix(backend): アンテナ等がポリシーで定められた上限を超えている場合、変更や追加ができないように (MisskeyIO#646)
a14e11d
@kabo2468 kabo2468 deleted the restrict-related-api branch June 22, 2024 11:24
u1-liquid pushed a commit that referenced this pull request Dec 22, 2024
@kakkokari-gtyih @zotanmew
fix(backend): ノートを連合する際にリモートユーザーのacctの大小文字を区別して処理している問題を修正 (misskey-d…
6718a54 
…ev#14880)

* fix: make sure outgoing remote mentions get resolved correctly if referenced with non-canonical casing (resolves #646)

* Update Changelog

* Update Changelog

* indent

---------

Co-authored-by: Laura Hausmann <laura@hausmann.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@u1-liquid u1-liquid u1-liquid approved these changes

Assignees
No one assigned
Labels
packages/backend:test packages/backend
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kabo2468 @kakkokari-gtyih @u1-liquid