PR

【1/24 15:00】海域イベント第5回 #3875

wada opened this at 2025-01-16T17:19:17; closed

Description

opened by wada at 2025-01-16T17:19:17; updated 2025-01-24T14:32:06

概要

仕様書

  • 海図の断片上限400
  • 海域の主討伐報酬更新
  • EX報酬天井解放

天井の表示周りで調整要望あれば対応
・結果モーダルの表示

他PR

【1/23 17:00】海域イベント告知画像 #3874
【1/27 15:00】海域イベント第5回 jolly/jrye-db#974

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

TODO

  • [x] 画像反映
  • [x] ログデータ移動
  • [x] dbデプロイ
  • [x] webデプロイ
  • [x] 戦闘デモ確認
  • [x] 告知設定

Comment

#38461 by wada at 2025-01-16T17:20:10; updated 2025-01-16T17:20:10

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

Comment

#38515 by kouho at 2025-01-20T17:26:21; updated 2025-01-20T17:26:21

@wada
40Fの【魔格】鬼国の酔剣ジュンのスキルの破断の気が二重で表示されていました。
!スクリーンショット 2025-01-20 17

kouho add_label at 2025-01-20T17:26:26: コメントあり

Comment

#38519 by wada at 2025-01-21T11:34:55; updated 2025-01-21T11:34:55

@kouho
> 40Fの【魔格】鬼国の酔剣ジュンのスキルの破断の気が二重で表示されていました。

修正しました。

Comment

#38524 by kouho at 2025-01-21T13:20:08; updated 2025-01-21T13:20:08

@wada
対応ありがとうございます。以下修正確認いたしました。
> 40Fの【魔格】鬼国の酔剣ジュンのスキルの破断の気が二重で表示されていました。
修正しました。

Comment

#38525 by kouho at 2025-01-21T13:32:27; updated 2025-01-21T13:32:27

@wada
1000Pt以上溜まった際にこちらの表示が出ますが、実際は1000Pt以上溜まっているように見えます。
1000Pt超えた分が溜まるのと溜まらないのではどちらが正でしょうか?
!スクリーンショット 2025-01-21 13

Comment

#38526 by kouho at 2025-01-21T13:49:13; updated 2025-01-21T13:49:13

@wada
エリア制覇報酬一覧のページが複数の役割があり、分かりづらくなっているように見えます。

エリアEX未到達の場合は表示は問題ないと思いますが、
エリアEX到達済みのページの表示が特に複雑になっていると思います。

エリアEX未到達の場合

  • ユーザーが次に獲得できる報酬の表示
  • クリアしていない各エリア毎の報酬の表示
  • EXステージランダム報酬の確認
  • EXステージ天井報酬の確認
  • EXステージ天井報酬の説明

エリアEX到達済みの場合

  • EXステージのポイントゲージ
  • EXステージ天井報酬の確認と交換の実行
  • ユーザーが次に獲得できる報酬の表示 + EXステージランダム報酬の確認
  • EXステージ天井報酬の説明

報酬一覧とEXステージ天井ポイントの表示&交換のページは分けることはできないでしょうか?

Comment

#38527 by kouho at 2025-01-21T13:52:05; updated 2025-01-21T13:52:05

@wada
こちらは改めて確認したところ、実際は1000Pt以上データ上は溜まりますが、表示上はMAXで交換時に0ptにリセットされました。
1000Pt超えた分が溜まらないようになっておりましたので実装上の問題はないです。
> 1000Pt以上溜まった際にこちらの表示が出ますが、実際は1000Pt以上溜まっているように見えます。
1000Pt超えた分が溜まるのと溜まらないのではどちらが正でしょうか?

Comment

#38528 by kouho at 2025-01-21T14:00:43; updated 2025-01-21T14:00:43

@wada
32767Ptも抱えたまま交換せず超える人はいないと思いますが、一応user_ocean_area_data.ex_ptをSMALLINTからINTに変更しておいていただきたいです。

Comment

#38533 by wada at 2025-01-21T14:30:49; updated 2025-01-21T14:30:49

@kouho
> エリア制覇報酬一覧のページが複数の役割があり、分かりづらくなっているように見えます。

おいおいレイアウト調整とのことです。

Comment

#38534 by kouho at 2025-01-21T14:53:34; updated 2025-01-21T14:53:34

@wada
かしこまりました。確認OKです。
> エリア制覇報酬一覧のページが複数の役割があり、分かりづらくなっているように見えます。
おいおいレイアウト調整とのことです。

kouho delete_label at 2025-01-21T14:53:39: コメントあり
kouho add_label at 2025-01-21T14:54:35: 皇甫 LGTM

Comment

#38540 by wada at 2025-01-21T14:55:03; updated 2025-01-21T14:55:03

@kouho
戦闘勝利後の結果モーダルが非表示になっていたので修正しました。

Comment

#38541 by kouho at 2025-01-21T15:01:52; updated 2025-01-21T15:01:52

@wada
結果モーダルの表示も確認OKです。
> 戦闘勝利後の結果モーダルが非表示になっていたので修正しました。

wada change_title at 2025-01-21T17:03:20: 【1/27 15:00】海域イベント第5回 【1/24 15:00】海域イベント第5回
wada merge at 2025-01-24T14:26:12: 【1/24 15:00】海域イベント第5回
wada close at 2025-01-24T14:26:12: Close
wada delete_branch at 2025-01-24T14:26:17: ocean_0124

Commit range

base6052347366d7c83b56f84d2c74fa45c8e69b35ee
headacb557cc3394acd015105617e3b6f70280931a25
base branchjolly/jr-web:master
head branchjolly/jr-web:ocean_0124

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed24 filesView patch
statusfile
modifiedlib/conf/class/OceanAreaConfig.class.php
modifiedlib/conf/define_y.inc
modifiedlib/public/Battle-ResultOceanArea.inc
deletedpublic_html/img/ocean_area/5/ab_extra.jpg
modifiedpublic_html/img/ocean_area/5/area_kouryakurank_header.jpg
modifiedpublic_html/img/ocean_area/5/areaitiran_header.jpg
modifiedpublic_html/img/ocean_area/5/cleaninghouse.jpg
modifiedpublic_html/img/ocean_area/5/count_killboss_header.jpg
addedpublic_html/img/ocean_area/5/ex_reward_bonus.jpg
modifiedpublic_html/img/ocean_area/5/header1.jpg
modifiedpublic_html/img/ocean_area/5/header2.jpg
deletedpublic_html/img/ocean_area/5/m_ab_extra.jpg
deletedpublic_html/img/ocean_area/5/m_area_kouryakurank_header.jpg
deletedpublic_html/img/ocean_area/5/m_areaitiran_header.jpg
deletedpublic_html/img/ocean_area/5/m_boss_battle_btn.jpg
deletedpublic_html/img/ocean_area/5/m_cleaninghouse.jpg
deletedpublic_html/img/ocean_area/5/m_count_killboss_header.jpg
deletedpublic_html/img/ocean_area/5/m_header1.jpg
deletedpublic_html/img/ocean_area/5/m_header2.jpg
modifiedpublic_html/js/JOLLYROGER.js
modifiedpublic_html/js/jr_main.min.js
modifiedpublic_html/template/OceanArea-Default.tpl
modifiedpublic_html/template/OceanArea-Ranking.tpl
modifiedpublic_html/template/header_sp.tpl
Patch body is stored as a separate file to keep this page lightweight.