JSON API mixes null and empty strings for missing URLs
まだ誰も着手していません。
評価
調査の方向性
まず、issue で言及されている list_archive_url、webscm_url、およびその他の nullable URL フィールドの API シリアライゼーションを追跡します。null と空の値の挙動が DRF と Patchwork のどちらによって制御されているかを特定し、nullable URL が一貫して null を生成すること、そしてその挙動が関連する API テストでカバーされていることを確認します。
索引モデルが issue の本文から書いたものです。
説明
This is in quite a few places.
list_archive_url is null (though it can be "" too! not sure what the correlation is), most other nullable URLs i.e. webscm_url are "".
I believe the best option is to have them all be null, avoiding cases where a JSON parser knows a field should contain a URL, finds a value (since it's not null), and can't parse it as a valid URL.
Also not sure if this is a DRF quirk or something Patchwork can control.
- 主要言語
- Python
- スター
- 317
- フォーク
- 91
- PR マージ指標
- 30日以内にマージされた PR はありません
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
getpatchwork/patchwork のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
getpatchwork/patchwork#668 ·
-
bug web-ui
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
getpatchwork/patchwork#546 · コメント 2 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 52/100
getpatchwork/patchwork#664 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
getpatchwork/patchwork#663 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
getpatchwork/patchwork#645 ·
getpatchwork/patchwork の issue をすべて見る
似ている issue
-
Add: hunch オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
AbdelStark/awesome-typesafe#104 ·
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
DiamondLightSource/dodal#2211 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
openml/openml-python#1749 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
sipyourdrink-ltd/bernstein#6191 ·