JSONAPI-Resources / JSONAPI-Resources/jsonapi-resources
Id is required for update of singular resource.
まだ誰も着手していません。
- 主要言語
- Ruby
- スター
- 2.3k
- フォーク
- 546
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
Hello. In my case I send put request to /profile route. And since this resource is singular, I don't need to provide id param. But, unfortunately, it throws A key is required error if I don't provide id(it works if I provide any random value as id).
It happens because of this line of code
https://github.com/cerebris/jsonapi-resources/blob/master/lib/jsonapi/request_parser.rb#L201
Are the any specific reasons for this behaviour?
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
lib/jsonapi/request_parser.rb の 201 行目付近を読み、単数形のリソース /profile への PUT リクエストがどのようにパースされるかを追跡してください。ID がない場合と任意の ID の場合を再現し、現在のキー要件が意図的なものかどうかを判断してください。期待される動作とその理由が明らかになり、動作を変更すべき場合は回帰テストが追加されていれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- ruby
- 領域
- api
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- 説明が足りない
- 初心者へのやさしさ
- 25/100