PR

【4/26】GWチームクエスト #3467

wada opened this at 2024-04-24T17:04:13; closed

Description

opened by wada at 2024-04-24T17:04:13; updated 2024-04-28T21:09:19

概要

https://docs.google.com/spreadsheets/d/13hqQC4Ap1X_JuUG9BMraBtKOAhG-v14GhpEo1WgpjNw/edit#gid=1729591901

チーム未所属のユーザは加算、報酬配布なし
限定クエストページを流用
他CPクエストとは併催不可
SP報酬は、CP終了後集計して配布予定

チーム移籍について

  • 移籍後のチームに消費GOLDは引き継がない
  • 移籍したユーザの消費GOLDは、チーム獲得量に加算される
  • CP報酬は一度だけ獲得可能、受取済み報酬は移籍しても獲得できない
  • 移籍後に移籍前のチームに戻った場合、消費GOLDはそのチームで過去消費した数値になる
  • SP報酬は、配布時にランキングTOPのユーザに配布

DB更新内容

【4/26】GWチームクエスト jolly/jrye-db#723

追加テーブル

#チームごとのユーザ消費量管理
CREATE TABLE `user_team_campaign_quest` (
  `team_no` int(11) unsigned NOT NULL,
  `user_no` int(11) unsigned NOT NULL,
  `campaign_no` tinyint(3) unsigned NOT NULL,
  `border_type` tinyint(3) unsigned NOT NULL,
  `border_count` bigint(20) unsigned NOT NULL,
  PRIMARY KEY (`team_no`,`user_no`,`campaign_no`,`border_type`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4;
#CPでの報酬受け取り管理
CREATE TABLE `log_team_campaign_quest_reward` (
  `id` int(11) unsigned NOT NULL AUTO_INCREMENT,
  `user_no` int(11) unsigned NOT NULL,
  `campaign_no` tinyint(3) unsigned NOT NULL,
  `quest_no` tinyint(3) unsigned NOT NULL,
  `reg_date` datetime NOT NULL,
  PRIMARY KEY (`id`),
  UNIQUE KEY `user_quest` (`user_no`,`campaign_no`,`quest_no`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4;

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

各テーブルを作成
master_team_campaign_questの期間を変更し確認

TODO

  • [x] チーム、海賊団表記統一
  • [x] ページ内テキスト調整
  • [x] チーム移籍後ランキング表示修正
  • [x] チーム移籍周り動作確認
  • [x] バナー表示
  • [x] 終了時対応(報酬獲得期間)
  • [x] js圧縮

Comment

#34295 by wada at 2024-04-24T17:04:55; updated 2024-04-24T17:04:55

@kouho
まだ残っている箇所ありますが、レビュー時間取れなさそうなので、先にPR作成します。
レビューお願いします。

Comment

#34357 by wada at 2024-04-25T16:21:08; updated 2024-04-25T16:21:08

@kouho
user_team_campaign_questテーブルの構成を変更しました。

kouho add_label at 2024-04-27T23:23:46: 皇甫 LGTM

Comment

#34389 by kouho at 2024-04-27T23:42:02; updated 2024-04-27T23:42:02

@wada
確認が遅れて申し訳ないです。
動作と実装は問題なさそうですが、細かい確認と細かい指摘です。

2点確認です。

  • 1点目

MVP報酬の配布集計は直接クエリで取得して配布予定でしょうか?

  • 2点目

MVP報酬の配布にも個人消費20万超えを判定に入れますか?
レアケースですが、MVP対象ユーザーが移籍して20万消費以下のユーザーが1位になる可能性がありそうでしたので確認です。

2点の指摘です。

  • 1点目

海賊団のデータが入っていない状態での表示で吹き出しと※のテキストが若干被っていました。(対応必須ではないレベルだと思います)
!スクリーンショット 2024-04-27 23

  • 2点目

注意事項のテキストで「は」が抜けてそうでした
消費したGOLD額新しい海賊団⇒消費したGOLD額は新しい海賊団

kouho add_label at 2024-04-27T23:42:06: コメントあり

Comment

#34391 by wada at 2024-04-28T09:19:55; updated 2024-04-28T09:19:55

@kouho
確認ありがとうございます。
指摘点修正しました。

MVP報酬は、CP終了後mysqlクエリで配布予定です。
条件については、チームで相談しましょう。

Comment

#34392 by kouho at 2024-04-28T11:10:26; updated 2024-04-28T11:10:26

@wada
ありがとうございます。指摘点修正確認OKです。条件についてのチーム相談もかしこまりました。

kouho merge at 2024-04-28T21:09:19: 【4/26】GWチームクエスト
kouho close at 2024-04-28T21:09:19: Close

Commit range

base9f2a0aeebb1bb1a3aa1fdf420e7bab9e1d726231
headbe786c449968d7d693853dd4c84291f522db70f6
base branchjolly/jr-web:master
head branchjolly/jr-web:team_cp_quest

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed37 filesView patch
statusfile
modifieddatabase/jrye.sql
modifiedlib/SubFunc.inc
modifiedlib/ajax/main_default.inc
modifiedlib/class/const/QuestConst.class.php
modifiedlib/class/controller/JrControllerSeasonPass.class.php
addedlib/conf/class/TeamCampaignQuestConfig.class.php
modifiedlib/models/Banner.php
addedlib/models/Team_campaign_quest.php
addedlib/models/dao/log_team_campaign_quest_reward.php
addedlib/models/dao/master_team_campaign_quest.php
addedlib/models/dao/master_team_campaign_quest_border.php
addedlib/models/dao/master_team_campaign_quest_reward.php
addedlib/models/dao/user_team_campaign_quest.php
modifiedlib/public/Quest-Campaign.inc
modifiedlib/public/Quest-Default.inc
modifiedlib/public/Quest-LoginBonus.inc
addedlib/public/Quest-TeamCampaign.inc
addedlib/public/Quest-TeamCampaignClear.inc
modifiedlib/public/Quest-TeamWarDetail.inc
addedpublic_html/img/banner/Campaign/banner_team_quest_1_1.jpg
addedpublic_html/img/banner/Campaign/banner_team_quest_1_2.jpg
addedpublic_html/img/info_jrye/3198/info1.jpg
addedpublic_html/img/info_jrye/3198/info2.jpg
addedpublic_html/img/info_jrye/3198/info3.jpg
addedpublic_html/img/info_jrye/3198/info4.jpg
modifiedpublic_html/index.php
modifiedpublic_html/js/JOLLYROGER.js
modifiedpublic_html/js/jr_main.min.js
modifiedpublic_html/template/Quest-Campaign.tpl
modifiedpublic_html/template/Quest-Default.tpl
modifiedpublic_html/template/Quest-LoginBonus.tpl
addedpublic_html/template/Quest-TeamCampaign.tpl
addedpublic_html/template/Quest-TeamCampaignNoTeam.tpl
modifiedpublic_html/template/Quest-TeamWarDetail.tpl
modifiedpublic_html/template/SeasonPass-Home.tpl
modifiedpublic_html/template/header_sp.tpl
modifiedpublic_html/template/parts_quest_tabs.tpl
Patch body is stored as a separate file to keep this page lightweight.