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

secure: true なエンドポイントの型が misskey-js に含まれていない #12598

Closed
zyoshoka opened this issue Dec 7, 2023 · 5 comments · Fixed by #12603
Closed

Comments

@zyoshoka
Copy link
Contributor

zyoshoka commented Dec 7, 2023

💡 Summary

#12434 にてバックエンドの生成する api.json から misskey-js の型が作られるようになりましたが、api.json の生成において secure: true なエンドポイント(たとえば i/update-email など)は省かれているので、このような(一部の?)エンドポイントの型が misskey-js に含まれなくなっています。これらについても misskey-js の型が生成されているべきです。

🥰 Expected Behavior

secure: true なエンドポイントが misskey-js の型に含まれている

🤬 Actual Behavior

secure: true なエンドポイント(i/update-email など)が misskey-js の型に含まれていない(元々含まれていた?)

📝 Steps to Reproduce

No response

💻 Frontend Environment

No response

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey: 2023.12.0-beta.2
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:
@zyoshoka zyoshoka added the ⚠️bug? This might be a bug label Dec 7, 2023
@samunohito
Copy link
Member

@syuilo
(gitのログ見たらしゅいろさんが実装されてたので)
なにか経緯はご存じだったりしますか…?非公開APIみたいな扱いでしょうか…

@syuilo
Copy link
Member

syuilo commented Dec 7, 2023

非公開APIだけどmisskey-jsに含めない理由はなさそう

@syuilo syuilo added packages/misskey-js and removed ⚠️bug? This might be a bug labels Dec 7, 2023
@samunohito
Copy link
Member

非公開APIとなると…

misskey-js → ふくめる
/api-doc(とそこから落とせるapi.json) → ふくめない

みたいな感じですかね?

@syuilo
Copy link
Member

syuilo commented Dec 7, 2023

api-docについても含めても良いんじゃないかしら

@syuilo
Copy link
Member

syuilo commented Dec 7, 2023

非公開という但し書きは必要

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants