PR

【10/19】シーズンパス4~7本家マージ #3055

wada opened this at 2023-09-01T13:39:18; closed

Description

opened by wada at 2023-09-01T13:39:18; updated 2023-10-19T14:43:15

概要

https://docs.google.com/spreadsheets/d/13P64IG2HBr76pQ-Dz33_qiapTRv16vBVMfn-iNrjF8o/edit#gid=0
シーズンパス5~7のマージ
・シーズン4はマスタ対応のみ
・シーズン5,6の報酬設定は先行してマージ済み
スキル処理のマージは別PR(シーズン6,7は未対応) #3049
本家のconfig設定などは入れないようにマージ
・SeasonPassConfigのシーズンごとの設定など
・マージ中に入っても無理に削除していない

ValhallaConfigでシーズン5,7の条件が追加されたが、現状必要性は高くないので今後削除する可能性あり

developマージを取り消し

developにマージする際は以下のrevertコミットをrevertすること

web

  • merge: 4ee0faf0d4f64e8a85cde7e3c2adddc381590229
  • revert: 31c7d9e0ffae35cddc65b27c96b7183575c5d515

db

  • merge: 7c766b3d40c98baac769e298b7545757c58939b9
  • revert: a5a27fe47a6ee360dd9669678aabbdea32ffea34

パイレーツパス

  • 7395【今日中】船の名前修正。そのほか文言修正など
  • 7481【12/1】パスミッション文言と港の修正。
  • 8019【6/10】パイレーツパス シーズン5

限定船処理の追加

  • 8030【即日】限定船に限り並び順決め打ち。
  • 8213【8/4】パイレーツパスシーズン6

限定船2の追加

  • 8373【9/29】パイレーツパスシーズン7

DB更新内容

【9/18】シーズンパス4~7本家マージ jolly/jrye-db#521

テスト方法、確認した内容

仲介所での船交換、造船、船関連の処理

デバッグとレビューを同時で進める予定

TODO

  • [x] デバッグ
  • [x] js圧縮
  • [x] 限定船非表示
  • [x] dbデプロイ
  • [x] webデプロイ

Comment

#30107 by wada at 2023-09-01T13:40:06; updated 2023-09-01T13:40:06

@kouho
確認お願いします。

wada change_title at 2023-09-04T18:59:40: 【9/18】シーズンパス4~7本家マージ 【9/22】シーズンパス4~7本家マージ
wada change_title at 2023-09-21T10:33:20: 【9/22】シーズンパス4~7本家マージ シーズンパス4~7本家マージ
wada change_title at 2023-10-04T15:04:09: シーズンパス4~7本家マージ 【10/11】シーズンパス4~7本家マージ

Comment

#30980 by kouho at 2023-10-11T15:16:26; updated 2023-10-11T15:16:26

@wada
1. ID 8213 SubFunc.inc S_GetShipCustomGold 関数内にcase 10065の定義も追加をお願いいたします。

2. TeamWarConfigのship_dead_timeは正常に移植されているでしょうか?こちらではGitBucket上の差分では表示されますがブランチをチェックアウトしてファイルを確認してみると、$this->_config['ship_dead_time'][10064]と$this->_config['ship_dead_time'][10065]は見つかりませんでした。

Comment

#30984 by wada at 2023-10-11T15:27:56; updated 2023-10-11T15:27:56

@kouho
すいません、こちらで確認した範囲ではどちらも対応しています。
ブランチが最新の状態になっているか確認してもらっても良いでしょうか?

Comment

#30985 by kouho at 2023-10-11T15:37:27; updated 2023-10-11T15:37:27

@wada
失礼いたしました。ブランチを一度削除してチェックアウトしたらどちらも正しく表示されましたのでこちら側の問題だったようです。確認OKです。

kouho add_label at 2023-10-11T15:37:30: 皇甫 LGTM

Comment

#31060 by wada at 2023-10-16T16:27:59; updated 2023-10-16T16:27:59

@kouho
本番反映するに当たって、限定船のリリース前非表示対応が抜けていました。
以下のコミットで対応しましたので、確認をお願いします。

  • 限定船非公開 f84903f206f4abca9dcfab01eb8eaecff7b18362
  • 素材を所持していたら港からの造船ボタンを表示しない 171a33d64739c6b394f5f7d860f8cf6ca372a855
wada delete_label at 2023-10-18T09:41:40: 皇甫 LGTM
wada change_title at 2023-10-18T10:03:30: 【10/11】シーズンパス4~7本家マージ 【10/19】シーズンパス4~7本家マージ
kouho add_label at 2023-10-18T18:04:02: 皇甫 LGTM

Comment

#31147 by kouho at 2023-10-18T18:05:26; updated 2023-10-18T18:05:26

@wada
確認OKです

wada merge at 2023-10-19T14:43:09: 【10/19】シーズンパス4~7本家マージ
wada close at 2023-10-19T14:43:09: Close
wada delete_branch at 2023-10-19T14:43:15: pirate_pass_origin

Commit range

base3e6f899a1e2f057c19b61a3a37278e21cd79b9b5
head171a33d64739c6b394f5f7d860f8cf6ca372a855
base branchjolly/jr-web:master
head branchjolly/jr-web:pirate_pass_origin

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed28 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/class/controller/JrControllerSeasonPass.class.php
modifiedlib/class/season_pass/SeasonPassQuestAction.class.php
modifiedlib/conf/class/CardPieceConfig.class.php
modifiedlib/conf/class/SeasonPassConfig.class.php
modifiedlib/conf/class/ShipBuildListConfig.class.php
modifiedlib/conf/class/TeamWarConfig.class.php
modifiedlib/conf/class/ValhallaConfig.class.php
modifiedlib/conf/define.inc
modifiedlib/public/CardPieceShop-Default.inc
modifiedlib/public/Commerce-Default.inc
modifiedlib/public/Ship-LimitedShip.inc
modifiedlib/public/Ship-SelectDetail.inc
addedpublic_html/img/main/st_icon_1007.gif
modifiedpublic_html/img/season_pass/reward_info/item_104_1.png
modifiedpublic_html/img/season_pass/reward_info/item_104_2.png
modifiedpublic_html/img/season_pass/reward_info/item_104_3.png
modifiedpublic_html/img/season_pass/reward_info/item_104_4.png
modifiedpublic_html/js/JOLLYROGER.js
modifiedpublic_html/js/jr_main.min.js
modifiedpublic_html/template/CardPieceShop-Default.tpl
modifiedpublic_html/template/Commerce-Default.tpl
modifiedpublic_html/template/Commerce-Home.tpl
modifiedpublic_html/template/SeasonPass-Home.tpl
modifiedpublic_html/template/Ship-CustomDetail.tpl
modifiedpublic_html/template/Ship-SelectDetail.tpl
modifiedpublic_html/template/header_sp.tpl
modifiedpublic_html/template/parts_cardpiece_info_modal.tpl
Patch body is stored as a separate file to keep this page lightweight.