PR

【5/23】ガチャ転生対応 #1845

wada opened this at 2022-05-20T13:48:21; closed

Description

opened by wada at 2022-05-20T13:48:21; updated 2022-05-23T10:42:45

概要

【5/18】ガチャ限定カード転生解放 #1834
【確認次第】転生ガチャIDによるフィルターを保持 #1838
【5/20】ガチャ画像転生対応 #1837

5/23 オリバーガチャリリース前に本番反映
リリース直後にキャッシュ対応

DB更新内容

【5/23】ガチャ転生対応 jolly/jrye-db#63

TODO

  • [x] dbデプロイ
  • [x] webデプロイ
  • [x] 告知設定(魅力紹介・トレード解禁更新)
  • [x] 戦闘デモ設定
  • [x] 管理画面確認

Comment

#19209 by wada at 2022-05-20T13:48:42; updated 2022-05-20T13:48:42

@kouho
確認お願いします。

wada refer at 2022-05-20T14:21:11: 1846:【5/23】ガチャ転生画像キャッシュ対応

Comment

#19215 by kouho at 2022-05-20T14:27:17; updated 2022-05-20T14:27:17

@wada
確認OKです

kouho add_label at 2022-05-20T14:27:22: 皇甫 LGTM

Comment

#19219 by kouho at 2022-05-20T15:06:10; updated 2022-05-20T15:06:10

@wada
確認OKと書いたのですが、こちらのPRは5/23 12:00ちょうどに反映させますか?
もしその前に反映させるとしたら転生でのガチャ画像関係の変更はキャッシュ対応の方のPRに含めた方がいいかもしれません。

Comment

#19220 by wada at 2022-05-20T15:33:13; updated 2022-05-20T15:33:13

@kouho
5/23 オリバーガチャリリース前に本番反映に本番反映します。
本番反映後、正しくデータが設定してあるか確認する時間を設けるためです。

> もしその前に反映させるとしたら転生でのガチャ画像関係の変更はキャッシュ対応の方のPRに含めた方がいいかもしれません。

どうしてそう思いましたか?

Comment

#19221 by kouho at 2022-05-20T16:57:48; updated 2022-05-20T16:57:48

@wada
なるほど。オリバーガチャは問題ありませんが、ネルソンガチャが気になりました。
ガチャ内のカードの転生のリリースが5/23 12:00なので開催中のネルソンガチャのカードの転生がリリースされる前に画像に転生や転生可能のマークが入っているのをユーザーが見る場合があるのではと思いました。
キャッシュ対応と同時に対応をしたらこの問題は起きないかなと。

Comment

#19235 by wada at 2022-05-23T10:03:53; updated 2022-05-23T10:03:53

@kouho
ありがとうございます。画像はキャッシュ対応のprに含めます。
マージ打ち消しコミットが含まれるので、以下の順に対応します。
1. 画像マージの打消し
2. このprをmasterマージ、本番反映
3. masterを画像キャッシュ対応prにマージ
4. 画像キャッシュ対応pr上で、マージ打消しコミットの打消し

wada merge at 2022-05-23T10:30:08: 【5/23】ガチャ転生対応
wada close at 2022-05-23T10:30:08: Close

Commit range

baseb34a47dc73e7829588da53b0d150505a73bfb6b8
head12f40fd1d9ecc84514442600bb509727bd94b596
base branchjolly/jr-web:master
head branchjolly/jr-web:release_gacha_limit_break

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed11 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/conf/define_y.inc
modifiedlib/public/Bar-Gacha.inc
modifiedlib/public/Demonstration-Battle.inc
modifiedlib/public/LimitBreak-List.inc
modifiedpublic_html/js/JOLLYROGER.js
modifiedpublic_html/js/jr_main.min.js
modifiedpublic_html/template/Bar-GachaReprint3.tpl
modifiedpublic_html/template/Bar-Gacha_old.tpl
modifiedpublic_html/template/LimitBreak-List.tpl
modifiedpublic_html/template/header_sp.tpl
Patch body is stored as a separate file to keep this page lightweight.