Description
opened by wada at 2023-03-17T14:34:58; updated 2023-03-23T09:38:35
概要
蒼玉交換 jolly/jr-cs-issue#338
画面作成してからだとレビュー時間が足りないので機能実装のみ
交換ログは蒼玉消費ログlog_user_blue_ballで代用
テスト方法、確認した内容
以下のURLにアクセスして召喚の書が増えるか、レイドが出現するか
#bb_change_numは交換したい蒼玉の数
/?M=RaidEvent&A=ExecBlueBallChange&bb_change_num=2
TODO
Comment
#26409 by wada at 2023-03-17T14:36:27; updated 2023-03-17T14:36:27
@kouho
画面作成が間に合わなそうなので、先に機能面のレビューをお願いします。
画面作成はデバッグまでに間に合えばこのPRに追加します。
Comment
#26416 by kouho at 2023-03-17T17:16:19; updated 2023-03-17T17:16:19
@wada
ありがとうございます。
NewiIslandConfigとRaidMonsterConfigの期間修正を巻き込んでしまっているので確認お願いいたします
wada change_title at 2023-03-20T15:14:26: 【3/20】蒼玉召喚の書交換
【3/21】蒼玉召喚の書交換
Comment
#26435 by wada at 2023-03-20T15:36:23; updated 2023-03-22T09:47:50
@kouho
交換ページ作成できたので、コミットしました。
対応箇所
メインページ
- 召喚の書アイコンの遷移先を交換ページに変更(レイド出現中は遷移しない)
イベントモーダル内のアイコンは蒼玉召喚を間違う可能性があるため遷移しない
- イベントモーダルに所持蒼玉を追加
- イベントモーダルの所持蒼玉リカバリで、期限付きの個数をまとめて表示
~~現在の仕様ではどちらかが必ず0なので~~
- イベントモーダルに交換ページへのボタンリンクを設置
アイテム使用
- イベント出現中以外でも蒼玉リカバリ(期限付き含む)使用可能
- アイテム使用後ページに所持蒼玉表示
- チーム未所属ユーザはリカバリを使用できない
交換ページ
- 蒼玉2個と召喚の書1個の交換対応
- 蒼玉2個未満は冒険への導線を表示
- 召喚の書が20個になりレイドが出現したらレイドへの導線を表示
複数個交換は、実装や確認に時間がかかるため対応していません。
できそうならdevelopマージ後に対応してもらって大丈夫です。
デバッグ日に合わせて、締め切りを3/21に変更しました。
Comment
#26439 by kouho at 2023-03-20T16:04:05; updated 2023-03-20T16:04:05
@wada
対応ありがとうございます。
log_team_raid_itemへのログの入り方が気になります。
レイド出現中に蒼玉を拾った場合log_team_raid_itemに元々入っていなかったのが今は入るようになっているように見えますので確認していただきたいです・
Comment
#26444 by kouho at 2023-03-20T18:32:04; updated 2023-03-20T18:32:04
Comment
#26446 by wada at 2023-03-20T18:49:36; updated 2023-03-20T18:49:36
@kouho
> log_team_raid_itemへのログの入り方が気になります。
レイド出現中に蒼玉を拾った場合log_team_raid_itemに元々入っていなかったのが今は入るようになっているように見えますので確認していただきたいです・
どの箇所でしょうか?
Comment
#26460 by kouho at 2023-03-22T16:11:36; updated 2023-03-22T16:15:31
@wada
> log_team_raid_itemへのログの入り方が気になります。
レイド出現中に蒼玉を拾った場合log_team_raid_itemに元々入っていなかったのが今は入るようになっているように見えますので確認していただきたいです・
>
どの箇所でしょうか?
すみません、これについては月曜日に伝えたように勘違いでした。
> $_df['MEMCACHE_NAME']を呼んでいるのでglobalに$_dfも定義お願いします。
> $_df['MEMCACHE_NAME']を呼んでいるのでglobalに$_dfも定義お願いします。
> 種類の違う期限付き蒼玉リカバリをまとめた個数表示するなら、
$_temp['blueball_item'][ItemConst::ID_BLUEBALL_RECOVERY_WED]['num'] += $weekday_item_manager->getAmount($item_no);
にした方が良いと思います。
これらの対応はどうしましょう?レイド実装の一環としてこちらで入れたほうが良いでしょうか?
Comment
#26464 by wada at 2023-03-22T17:44:35; updated 2023-03-22T17:44:35
@kouho
以下対応しました。
> $_df['MEMCACHE_NAME']を呼んでいるのでglobalに$_dfも定義お願いします。
$_df['MEMCACHE_NAME']を呼んでいるのでglobalに$_dfも定義お願いします。
種類の違う期限付き蒼玉リカバリをまとめた個数表示するなら、
$_temp['blueball_item'][ItemConst::ID_BLUEBALL_RECOVERY_WED]['num'] += $weekday_item_manager->getAmount($item_no);
にした方が良いと思います。
kouho add_label at 2023-03-22T18:31:40: 皇甫 LGTM
Comment
#26476 by kouho at 2023-03-22T18:31:53; updated 2023-03-22T18:31:53
wada merge at 2023-03-23T09:38:35: 【3/21】蒼玉召喚の書交換
wada close at 2023-03-23T09:38:35: Close