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

fix #3303 カスタムコンテンツ 並び順の選択肢の中に「公開日付」が含まれていない問題を解決 #3304

Merged
merged 5 commits into from
Apr 3, 2024

Conversation

katokaisya
Copy link
Collaborator

@katokaisya katokaisya commented Apr 2, 2024

@ryuring
@kaburk
並び順フィールドのリストに
published 公開日付を追加しました。
配置もできる限り上位が望ましいと思われたので、Noのすぐ下に入れています。(フロント側の一覧表示で、作成日順で表示するよりも明らかに使用頻度が高くなるため。)
お手数ですが、ご確認お願いできますでしょうか?

@katokaisya katokaisya added the Review レビュー要 label Apr 2, 2024
@katokaisya
Copy link
Collaborator Author

テスト通りました。
やはりテストコードにデフォルトカラムのpublishedがなかったのが原因でした。

@ryuring ryuring merged commit 31ec015 into baserproject:dev-5 Apr 3, 2024
1 check passed
@ryuring
Copy link
Collaborator

ryuring commented Apr 3, 2024

@katokaisya 確かにそうですね、ありがとうございます。マージしました。

@momofff momofff added this to the 5.0.12 milestone Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Review レビュー要
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants