Description
opened by wada at 2021-11-17T18:31:04; updated 2026-01-28T10:03:38
概要
トレード解禁の自動化に合わせて、解禁状態を master_card.trade_flg から専用テーブル参照に変更
解禁状態をわかりやすくするため、カード詳細とアルバムのトレード画像をタップして解禁状態をダイアログで表示
解禁状態&コメント
1. 未登録:トレード解禁は予定されていません
2. イベント中:イベント名 終了から約1ヶ月でトレード解禁されます
3. 30日以内:解禁日時 にトレード解禁予定です
4. 解禁済み:トレードできます!トレードQ&Aのリンク
5. 例外1:このカードはトレードできません(例:とくぱん、ティアリン)
6. 例外2:船はトレードできません
ブランチ
web: trade_update
db: trade_update
トレード解禁日の設定
管理画面の自動トレード解禁から設定
テスト方法、確認した内容
master_trade_release_time を操作して表示を変更
official_info_id: 登録時に作成される告知IDevent_name: ダイアログ内に表示されるイベント名trade_card_type: カードのタイプ(現状特に意味なし)trade_release_time: 解禁日時
TODO
お知らせが同時作成されるため、トレード解禁は開発の管理画面で登録する
wada assign at 2021-11-17T18:31:08: Not assigned:wada
wada change_title at 2021-11-19T10:19:21: [WIP]トレード解禁状態の表示
【11/24】トレード解禁状態の表示
wada add_label at 2021-11-19T10:24:44: レビュー待ち
Comment
#15364 by wada at 2021-11-19T10:47:59; updated 2021-11-19T10:47:59
@iwamoto @kouho
レビューお願いします。
Comment
#15375 by kouho at 2021-11-19T14:19:59; updated 2021-11-19T14:40:24
@wada
1. master_trade_release_timeには既にトレード解禁をされているカードを全てデータを入れる想定でしょうか?
2. アルバムからのトレード解禁済みのカードをみたさいに、トレードについてのリンクがおかしくなってました。 a href="class="css_font11"
Comment
#15376 by kouho at 2021-11-19T14:46:07; updated 2021-11-19T14:46:07
解禁状態とコメントについて、その他特に問題ないと思います。
Comment
#15382 by wada at 2021-11-19T15:23:17; updated 2021-11-19T15:23:17
@kouho
トレードリンク修正しました。
master_trade_release_timeにトレード解禁済みのデータは全て入っています。
正確には、トレード解禁の告知を開発環境で作成しているカードを登録しています。
告知が未作成の直近追加されたカードについては、管理画面に当PRをマージ後、管理画面からトレード解禁を設定します。
これは、登録時にトレード解禁の告知が自動で作成されるため、
・先にマスタに登録することで、告知を手作業で作成するコスト
・先に告知を登録することで、告知が2重登録される問題
を避けるためです。
Comment
#15390 by kouho at 2021-11-19T16:07:33; updated 2021-11-19T16:07:33
@wada
トレードについてのリンクを修正確認しました。OKです。
master_trade_release_timeのデータにつきましてもかしこまりました。
kouho add_label at 2021-11-19T16:07:36: 皇甫 LGTM
Comment
#15394 by iwamoto at 2021-11-19T16:57:22; updated 2021-11-19T16:57:22
@wada
確認しました、コード等問題ないと思います
気になった点で
master_trade_release_time.ymlのcard_noにauto_incrementが付いていますが、なくてもいいのではと思いました。
iwamoto add_label at 2021-11-19T16:57:27: 岩本 LGTM
Comment
#15396 by wada at 2021-11-19T17:14:54; updated 2021-11-19T17:14:54
@iwamoto
そうですね。auto_increment削除しました
Comment
#15397 by iwamoto at 2021-11-19T18:48:23; updated 2021-11-19T18:48:23
wada merge at 2021-11-22T12:11:29: 【11/24】トレード解禁状態の表示
wada close at 2021-11-22T12:11:29: Close
wada refer at 2021-11-30T11:25:44: 1351:【12/1】自動トレード解禁リリース