misskey
misskey copied to clipboard
misskey-dev
広告の曜日を設定できるように
What
広告に曜日の設定をできるように修正を行った
全てにチェックが入っていない場合も表示するようにしている
Why
毎週何曜日にこの広告を優先度○で出してほしい等の要望がある
Additional info (optional)
ローカル環境にてテスト済み
下記確認 ・チェック後のupdateにて設定値が入っていること ・チェックを外した際にuptdateにて規定値が入っていること ・ラベルクリック時にクリックした項目の値が入っていること(保存時に確認 ・チェックが入っていない場合の広告は表示されない(別曜日にてチェックが入っている場合 ・チェックが入っている曜日の広告が表示される ・どの曜日にもチェックが入っていない場合は表示される
Codecov Report
Merging #10095 (8082a08) into develop (209d8b4) will increase coverage by
48.64%. The diff coverage is86.36%.
@@ Coverage Diff @@
## develop #10095 +/- ##
============================================
+ Coverage 24.72% 73.36% +48.64%
============================================
Files 705 809 +104
Lines 65224 77526 +12302
Branches 2303 5403 +3100
============================================
+ Hits 16125 56880 +40755
+ Misses 49099 20646 -28453
| Impacted Files | Coverage Δ | |
|---|---|---|
| ...ackend/src/server/api/endpoints/admin/ad/create.ts | 74.54% <66.66%> (+74.54%) |
:arrow_up: |
| ...ackend/src/server/api/endpoints/admin/ad/update.ts | 75.00% <66.66%> (+75.00%) |
:arrow_up: |
| packages/backend/src/server/api/endpoints/meta.ts | 97.99% <91.66%> (+97.99%) |
:arrow_up: |
| packages/backend/src/models/entities/Ad.ts | 92.75% <100.00%> (+0.32%) |
:arrow_up: |
... and 638 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
![codecov[bot] avatar](https://avatars.githubusercontent.com/in/254?v=4 "Published by a pub.dev verified publisher"){kind=small}
広告に曜日を紐づけたいケースがあんまり理解できていない (男性のみに見せたいとか、若年層のみに見せたいとかはあるけど、この曜日だけに見せたい、というのはどういう意図だろう)
広告に曜日を紐づけたいケースがあんまり理解できていない
例えばユーザーが多い休日のみをターゲットにするとか、毎週その曜日にyoutube等に動画がアップされるのでその日だけ告知したい等がある
(現在ioでも毎週日曜日に指定した広告を流してほしいとのこともあったので
サーバーのタイムゾーン周りで不可解な挙動になりそうなのでタイムゾーン指定までさせたほうがいいかもしれない
これは今の修正みたいな感じで指定するので合ってる？
いや、サーバーのタイムゾーンがどこかわからない (基本 UTC な気はするが場合によっては普通に現地時間=日本だったら Asia/Tokyo だったりする) ので管理者が曜日を設定する時にタイムゾーンも設定できたほうがいいという話をしたかった
広告ごとにタイムゾーン決めてあったほうが英語圏とかでは嬉しい気がするけどそういうこと言い出すとクエリが面倒になる気がしてきた (面倒だったらとりあえずサーバーの現在タイムゾーン表示 & 変更するには TZ 環境変数で指定せいってUIに書いておくでいい気もする)
例えばユーザーが多い休日のみをターゲットにするとか
曜日を選択できるようにするということは、逆にユーザーの少ない平日のみをターゲットにしたいユースケースも存在するということ？
