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

misskey pageページでピン留めを解除することができない不具合 #10950

Closed
nan8ra opened this issue Jun 5, 2023 · 3 comments
Labels
🐛Bug Unexpected behavior packages/backend Server side specific issue/PR

Comments

@nan8ra
Copy link

nan8ra commented Jun 5, 2023

💡 Summary

(ノート、note、ピン留めで検索してヒットしなかったのでissueさせていただきます。)
自分で作成したmisskeyページをピン留めした後、解除を押すと「パラメータエラー リクエストパラメータに問題があります。通常これはバグですが、入力した文字数が多すぎる等の可能性もあります。」と表示され解除することができない。

🥰 Expected Behavior

ピン留めを解除と書かれているため正常に動作する場合は解除が行われるべきと考えます。

🤬 Actual Behavior

「パラメータエラー リクエストパラメータに問題があります。通常これはバグですが、入力した文字数が多すぎる等の可能性もあります。」と表示され、リクエストは飛んでいるのか確認するため更新を押した場合でも解除されていない表示(ピン留めを解除)がされます。

📝 Steps to Reproduce

1.エラーとなっているパラメータの修正

📌 Environment

misskey.io/misskey.cfで確認済み

Misskey version:13.12.2
Your OS:Windows 10 Pro
Your browser:Firefox13.12.2

@nan8ra nan8ra added the ⚠️bug? This might be a bug label Jun 5, 2023
@chocolate-pie
Copy link
Contributor

こうすれば直ったかも

diff --git a/packages/backend/src/server/api/endpoints/i/update.ts b/packages/backend/src/server/api/endpoints/i/update.ts
index d10f690a3..8f5e6177c 100644
--- a/packages/backend/src/server/api/endpoints/i/update.ts
+++ b/packages/backend/src/server/api/endpoints/i/update.ts
@@ -146,7 +146,7 @@ export const paramDef = {
 		alwaysMarkNsfw: { type: 'boolean' },
 		autoSensitive: { type: 'boolean' },
 		ffVisibility: { type: 'string', enum: ['public', 'followers', 'private'] },
-		pinnedPageId: { type: 'string', format: 'misskey:id' },
+		pinnedPageId: { type: 'string', format: 'misskey:id', nullable: true },
 		mutedWords: { type: 'array' },
 		mutedInstances: { type: 'array', items: {
 			type: 'string',

@nan8ra
Copy link
Author

nan8ra commented Jun 5, 2023

迅速なご対応ありがとうございます。
確認いたしました。本件はクローズさせていただきます。

@nan8ra nan8ra closed this as completed Jun 5, 2023
@chocolate-pie
Copy link
Contributor

いや、違います。この変更はコミットされていないので閉じないでください。

@syuilo syuilo reopened this Jun 5, 2023
@syuilo syuilo added 🐛Bug Unexpected behavior packages/backend Server side specific issue/PR and removed ⚠️bug? This might be a bug labels Jun 5, 2023
@tamaina tamaina closed this as completed in 565c502 Jun 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛Bug Unexpected behavior packages/backend Server side specific issue/PR
Projects
None yet
Development

No branches or pull requests

3 participants