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

Management API 対応 #37

Open
albno273 opened this issue Sep 26, 2022 · 10 comments
Open

Management API 対応 #37

albno273 opened this issue Sep 26, 2022 · 10 comments

Comments

@albno273
Copy link

日頃からお世話になっております。

本番環境では公開済みのコンテンツのみを表示したいが、テスト環境では下書き状態のコンテンツも含めて表示し、一覧ページや詳細ページなどサイト全体の表示を確認したいという状況に直面しましたので、パラメータなどで叩く先を Management API に切り替えられるようにしていただけると大変ありがたいです…!

@dc7290
Copy link
Member

dc7290 commented Sep 26, 2022

@albno273

Issueを発行してくださりありがとうございます!!!
確かにマネジメントAPIについてもSDKで提供できる方が良さそうですね・・・!

ただ今回の場合ですとAPIキーの「下書きの全取得」で要件満たせるかもです!
そちらもご確認頂けると幸いです。

@albno273
Copy link
Author

@dc7290
返信ありがとうございます。
確かにそれでいけそうです…!試してみます!

@hori-ryota
Copy link

hori-ryota commented May 28, 2023

@dc7290 こちら実装予定はありますか?

公開予定を設定しているコンテンツに対し処理を行いたいのですが、公開予定を設定しているコンテンツを判別する方法がManagement APIしかないため利用を検討しております。

方針として

  • インターフェースは増やさず、 BASE_DOMAINbaseUrl をリクエスト時に差し替え可能にする
  • Management API用のインターフェースを整備する

の2パターンがありそうですが、ベータなので簡易的に前者で対応し、正式対応までには後者を検討する、という進め方はいかがでしょうか。
レスポンスの形式が微妙に異なるので最終的にはちゃんとインターフェースを整える必要がありそうですが、contentsのmetadata取得系をいったん利用してみる分には前者でコスパよく利用できるように見えました。

@hori-ryota
Copy link

baseUrlを差し替える方針で #56 にて実装してみました。

@dc7290
Copy link
Member

dc7290 commented May 30, 2023

@hori-ryota

コメント、並びにPRの作成ありがとうございます!

結論から言うと、ManagementAPIについて社内でもSDKの方針について定まっていないんですよね・・・
具体的にはこのSDKに取り込むのか、別ライブラリとして扱うのかなどになります。

つきましてはいったんこのPRについては閉じさせてもらって、通常のfetchを使用してもらう形になりそうです💦

@hori-ryota
Copy link

@dc7290 返信ありがとうございます。

承知いたしました。いったんfetchで実装はしたのでそちらを利用してみます。

一方で当リポジトリのNode.jsバージョンが14系(サポート終了済み)だったりでsdkのメンテナンス状況が気になっています。
もしsdk利用を推奨というわけではなさそうでしたら別途作成しようと思いますが、当面の状況はいかがでしょうか。

(と書いていて思ったのですが、Discord入ったほうが良さそうですね、後でお邪魔します!)

@dc7290
Copy link
Member

dc7290 commented May 30, 2023

おっと!
Nodeのバージョンが低い件は単純に見逃しですね💦

ぜひPRとか作ってもらえると大変助かります😭

@hori-ryota
Copy link

了解です!↓のmswの件が落ち着くのを待つかもですが、手を付けられたらつけてみます!

mswjs/msw#1388

@tecsoc
Copy link

tecsoc commented Sep 1, 2023

@dc7290
コンテンツのステータスを取得するため、Management APIを使用したいと考えています。
こちらのIssueでのやりとりからかなりの間が経過していると思うのですが、まだ対応方針が決まっていないのでしょうか?最新情報を共有頂けると幸いです

@dc7290
Copy link
Member

dc7290 commented Mar 14, 2024

@tecsoc
返信遅れて申し訳ありません🙇‍♂️
現在、ちょうど画像アップロードAPIへの対応で、マネジメントAPIへの対応をしております。
コンテンツのステータスの取得についてはまだ未定とはなりますが、一旦ご報告まで!

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

No branches or pull requests

4 participants