PR

【5/18】ヤマトコラボ:勝利の証設定 #794

wada opened this at 2021-05-10T18:56:46; closed

Description

opened by wada at 2021-05-10T18:56:46; updated 2024-07-12T16:57:05

issue #753

Branch

web: fix_card_piece
db: yamato_skill

更新内容

  • コラボの勝利の証を実装
  • コラボ船の設計図を仲介屋に追加

交換制限、表示期限を設定

  • 設計図による造船に対応
wada change_title at 2021-05-13T10:55:24: [WIP]【5/14】ヤマトコラボに伴う仲介屋機能の更新 [WIP]【5/18】ヤマトコラボ:勝利の証設定
wada change_priority at 2021-05-13T10:58:34: No priority:very high

Comment

#10901 by wada at 2021-05-13T21:09:18; updated 2021-05-13T21:09:18

@iwamoto @kouho
レビューお願いします。

  • コラボの勝利の証で仲介屋のアイテムを交換できる
  • 今まで通り通常の勝利の証が交換できる
  • 交換した設計図で造船できる

画像などが不足していると思うので、確認用のブランチを作成して、
yamato_skillfix_card_pieceをマージすると確認しやすいです。

wada change_title at 2021-05-13T21:09:28: [WIP]【5/18】ヤマトコラボ:勝利の証設定 【5/18】ヤマトコラボ:勝利の証設定
wada add_label at 2021-05-13T21:09:35: レビュー待ち

Comment

#10989 by iwamoto at 2021-05-17T16:18:08; updated 2021-05-17T16:29:48

@wada
画像ではわかりにくいかもしれませんが、
証を所持している場合にグレースケールになってしまっています。

上記の影響かと思いますが、
「依頼する」からモーダルウィンドウが表示され、「はい」ボタンから交換ができません
!仲介屋-2

Comment

#10992 by kouho at 2021-05-17T18:20:06; updated 2021-05-17T18:20:06

@wada
!スクリーンショット 2021-05-17 18

カード画像右側の説明文で2箇所

  • 「3乱射」のスキルで→「乱射」のスキルで
  • 「山本玲」のスキルで攻撃力や速さがアップ→「山本玲」のスキルで攻撃力がアップ

速さも確かに上がりますが、コスモタイガーのみに対しての効果ではないので修正した方が良いかと思いました。

Comment

#10994 by kouho at 2021-05-17T18:42:20; updated 2021-05-17T18:42:20

@wada
!スクリーンショット 2021-05-17 18

仲介屋では、勝利の証と消費アイテムを引き換えに、海賊カードを手に入れることができます。
海賊カード以外も交換できるようになるためこの文言を削除した方が良いかと思います。
また、仲介屋から獲得した設計図の確認方法を注意文言に記載した方がわかりやすいかと思いました。

記載例

★交換した設計図は「港」→「限定船を造船する」より確認できます。

Comment

#10998 by wada at 2021-05-18T11:32:08; updated 2021-05-18T11:32:08

@kouho
コスモタイガーの紹介画像更新しました。
スキル説明については、魅力紹介の場でもあるので、効果がある物は全て記載したいと思います。
ルシエ紹介のテキスト修正しました。
注意文言については、仲介屋ページは一旦このままで、詳細はお知らせで補足します。

Comment

#10999 by wada at 2021-05-18T11:33:40; updated 2021-05-18T11:33:40

その他に設計図専用テキストと船のステータス表記を修正しました。
<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1621305147154UPM6jbBZuz" width="300">

Comment

#11002 by iwamoto at 2021-05-18T12:07:41; updated 2021-05-18T12:10:14

@wada
上記lib/models/Card_piece.phpは気になった点です、余裕があれば対応していただければと思います。

・ コラボの勝利の証で仲介屋のアイテムを交換できる
-> できました。
・今まで通り通常の勝利の証が交換できる
-> 問題なく交換できました。
・交換した設計図で造船できる
-> 造船/解体問題なく出来ました。

他は問題ないように思いました。

iwamoto add_label at 2021-05-18T12:10:38: 岩本 LGTM

Comment

#11006 by wada at 2021-05-18T12:21:51; updated 2021-05-18T12:22:33

@iwamoto
関数の間違いについて対応しました。うすうす

Comment

#11007 by iwamoto at 2021-05-18T12:30:20; updated 2021-05-18T12:45:04

@wada
すみません、ありがとうございます
こちら勘違いかと削除してしまいましたが…
一応こちらの環境の問題かもしれません
設計図獲得時の文言が画像のように
「が仲間になりました!を手に入れました!」となっていますので確認をお願いします。

エラー等は発生していないように思います。
!仲介屋獲得

Comment

#11009 by kouho at 2021-05-18T15:31:13; updated 2021-05-18T15:34:25

@wada
・コスモタイガーの紹介画像
・ルシエ紹介のテキスト
修正確認しました

・スキル説明
・注意文言
かしこまりました。

設計図専用テキストと船のステータス表記
問題ないと思います。

また、岩本さん指摘の仲介時の設計図獲得文言については修正済みかもですが、
こちらでは確認できませんでした。

kouho add_label at 2021-05-18T15:34:39: 皇甫 LGTM
wada close_comment at 2021-05-19T10:02:47: マージ完了
wada refer at 2021-05-21T19:43:18: 824:【5/24】ヤマトコラボリリース

Commit range

base773889cf85d47d56841e25ec049023392e416864
headba9ed3d864373e5a53728603a099265986cc702f
base branchjolly/jr-web:develop
head branchjolly/jr-web:fix_card_piece

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed55 filesView patch
statusfile
modifiedlib/conf/class/CardPieceConfig.class.php
addedlib/conf/class/ShipBuildListConfig.class.php
modifiedlib/mobile/CardPieceShop__CardDetail.inc
modifiedlib/mobile/CardPieceShop__Check.inc
modifiedlib/mobile/CardPieceShop__Default.inc
modifiedlib/mobile/CardPieceShop__Result.inc
modifiedlib/mobile/Present__Default.inc
modifiedlib/mobile/Present__History.inc
modifiedlib/mobile/Ship__Default.inc
modifiedlib/mobile/Ship__LimitedShip.inc
modifiedlib/mobile/Ship__MakeComposite.inc
modifiedlib/mobile/Ship__SelectDetail.inc
modifiedlib/models/Card_piece.php
modifiedlib/public/CardPieceShop-Default.inc
modifiedlib/public/Ship-Default.inc
modifiedlib/public/Ship-MakeComposite.inc
modifiedmobile_html/template/CardPieceShop__CardDetail.tpl
modifiedmobile_html/template/CardPieceShop__Check.tpl
modifiedmobile_html/template/CardPieceShop__Default.tpl
modifiedmobile_html/template/CardPieceShop__Result.tpl
addedpublic_html/img/card_piece/card_detail/10063_1.png
addedpublic_html/img/card_piece/card_detail/10063_2.png
addedpublic_html/img/card_piece/card_detail/10064_1.png
addedpublic_html/img/card_piece/card_detail/10064_2.png
modifiedpublic_html/img/card_piece/card_detail/553_1.png
renamedpublic_html/img/card_piece/shop/icon/1.png -> public_html/img/card_piece/icon/1.png
addedpublic_html/img/card_piece/icon/7.png
addedpublic_html/img/card_piece/icon/8.png
renamedpublic_html/img/card_piece/shop/icon/general.png -> public_html/img/card_piece/icon/general.png
renamedpublic_html/img/card_piece/shop/icon/general2.png -> public_html/img/card_piece/icon/general2.png
addedpublic_html/img/card_piece/shop/limited.png
addedpublic_html/img/card_piece/shop/paper_limit_time.png
addedpublic_html/img/item/card_piece/120/7.jpg
addedpublic_html/img/item/card_piece/120/8.jpg
addedpublic_html/img/item/card_piece/150/7.jpg
addedpublic_html/img/item/card_piece/150/8.jpg
addedpublic_html/img/item/card_piece/58/7.jpg
addedpublic_html/img/item/card_piece/58/8.jpg
addedpublic_html/img/item/card_piece/60/7.jpg
addedpublic_html/img/item/card_piece/60/8.jpg
addedpublic_html/img/item/card_piece/70/7.jpg
addedpublic_html/img/item/card_piece/70/8.jpg
addedpublic_html/img/main/st_icon_1004.gif
addedpublic_html/img/main/st_icon_1005.gif
addedpublic_html/img/material/120/1004.jpg
addedpublic_html/img/material/120/1005.jpg
addedpublic_html/img/material/150/1004.jpg
addedpublic_html/img/material/150/1005.jpg
addedpublic_html/img/material/60/1004.jpg
addedpublic_html/img/material/60/1005.jpg
addedpublic_html/img/material/70/1004.jpg
addedpublic_html/img/material/70/1005.jpg
modifiedpublic_html/js/JOLLYROGER.js
modifiedpublic_html/template/CardPieceShop-Default.tpl
addedpublic_html/template/parts_cardpiece_info_modal.tpl
Patch body is stored as a separate file to keep this page lightweight.