fix: profile update api schema to allow null language field #1

Closed
sup39 wants to merge 1 commit from fix/profile-update-api-schema into main
Owner

言語設定機能の実装によりプロフィールの言語を設定していない時に更新できない問題に関して調査したところ、関連するAPIのparamDeflang fieldの定義が原因だと判明しました。

Ajvのdocumentによると、nullableのenum fieldは明示的にnullをenumのリストに入れる必要があります。

If you use enum or const keywords, "nullable": true would not extend the list of allowed values - null value has to be explicitly added to enum (and const would fail, unless it is "const": null)

言語設定機能の実装により[プロフィールの言語を設定していない時に更新できない問題](https://hoshipaso.net/notes/9npy673rqbcz8q2u)に関して調査したところ、関連するAPIの`paramDef`の`lang` fieldの定義が原因だと判明しました。 [Ajvのdocument](https://ajv.js.org/json-schema.html#nullable)によると、nullableのenum fieldは明示的に`null`をenumのリストに入れる必要があります。 > If you use enum or const keywords, `"nullable": true` would not extend the list of allowed values - `null` value has to be explicitly added to `enum` (and `const` would fail, unless it is `"const": null`)
sup39 added 1 commit 2023-12-27 17:47:49 +09:00
Author
Owner
Resolved: https://post.naskya.net/notes/9nr4kus7u3m9ighc
sup39 closed this pull request 2023-12-27 18:17:15 +09:00
sup39 deleted branch fix/profile-update-api-schema 2023-12-27 18:20:51 +09:00
Author
Owner
https://forgejo.sup39.dev/sup39/firefish/commit/f676cc1f19f7abd19bc1733d9e31296fd8fc5ceb

Pull request closed

Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: sup39/firefish#1
No description provided.