PR

【11/24】トレード解禁状態の表示 #1313

wada opened this at 2021-11-17T18:31:04; closed

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: 登録時に作成される告知ID
  • event_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.ymlcard_noauto_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
確認しました、問題ないと思います

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】自動トレード解禁リリース

Commit range

base2fc9be4e678a3a6ca385adfc5c143f2af75967a9
head97d0220d7bb27ffa1edf355f22f5d12793ad0993
base branchjolly/jr-web:develop
head branchjolly/jr-web:trade_update

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed19 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/SubFunc_tournament.inc
modifiedlib/cpanel/m_card_data.inc
modifiedlib/cpanel/m_card_search_result.inc
addedlib/models/Trade_release.php
addedlib/models/dao/master_trade_release_time.php
modifiedlib/public/Card-AlbumDetail.inc
modifiedlib/public/Card-Detail.inc
modifiedlib/public/Card-ListGroup.inc
modifiedlib/public/Friend-Trade.inc
modifiedlib/public/Friend-TradeApplication.inc
modifiedlib/public/Friend-TradeCheck.inc
modifiedpublic_html/css/design.css
addedpublic_html/img/common/btn_trade.png
addedpublic_html/img/common/btn_trade2.png
modifiedpublic_html/js/JOLLYROGER.js
modifiedpublic_html/template/Card-AlbumDetail.tpl
modifiedpublic_html/template/Card-Detail.tpl
modifiedpublic_html/template/Friend-Trade.tpl
Patch body is stored as a separate file to keep this page lightweight.