【6/7】スタンプラリー #858

仕様
https://docs.google.com/spreadsheets/u/2/d/1OCzfQ5AN8KYRa-vhJUaApPI_dMg6fzEj-8UZDff1B_Q/htmlview#

ミッション数が5から9個に変更になったため、テーブル構成を変更
・master_yamada_stamp_rally
・user_yamaada_stamp_rally2（追加）

本番反映時、webから更新しないとエラーが発生する

web: stamp_rally_6
db: stamp_rally_6

#テーブル追加
CREATE TABLE `user_yamada_stamp_rally2` (
  `user_no` int(11) unsigned NOT NULL,
  `rally_id` int(11) NOT NULL,
  `stamp_id` int(11) NOT NULL,
  `count` tinyint(4) NOT NULL DEFAULT '0',
  `send_api` tinyint(4) NOT NULL DEFAULT '0',
  `date` int(11) NOT NULL,
  PRIMARY KEY (`user_no`,`rally_id`,`stamp_id`,`date`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4;

@iwamoto @kouho
レビューお願いします。
処理をテーブルごと変更したので、スタンプ更新までの1連の動作を確認お願いします。

開催期間は、master_yamada_campaignの期間を変更してください。

@wada
確認中ですが、2点ほど。

1点目 ヤマダキャンペーン期間の判定について
スタンプラリーに限らないですが、master_yamada_campaignのget_active_campaigns関数でNOW() < %sと判定していますが、end_dateをxxxx-xx-xx 23:59:59 にするならNOW() <= %s にした方が良いかと思いました。

2点目 ベルを鳴らそうのミッションでの回数について
海賊団チャットからベルを鳴らした場合、user_yamada_stamp_rally2のcountの増え方おかしいことがあり、
一度に5増えたことがありました。(冒険で出会った海賊でベルを鳴らす動作は未確認)
↑
単純に$stamp_bell_count=1にしたらどうかと思いました。

冒険で海賊と出会ったベルを鳴らした場合はスタンプ対象外ですかね、スタンプの詳細説明文的にも
もし対象外とするなら海賊団チャットから祝福のベルを鳴らそうにスタンプのタイトルも変えた方がいいかもしれません。

その他についてはこちらで確認した限り各条件を踏んでまた、達成する毎に
user_yamada_stamp_rally2が更新されておりOKかと思います。

@kouho
海賊団チャットからベルを鳴らすと、過去のベルを一括で鳴らすので、未読分一気にカウントを増やすようにしています。
鳴らすときに未読分がどれくらいあったか確認して欲しいです。

終了時間について、0時に修正しました。
すいません、バナー画像とお知らせについては、後ほど対応します。

@wada
海賊団チャットからのベルについて了解です。確かに溜まっていた分countが入っていました。
終了時間の修正確認しました。
バナー画像とお知らせについてかしこまりました。

あと、すみません。メンションをつけ忘れてしまっていたのですが、
冒険で海賊と出会ってベルを鳴らした場合についても確認お願いいたします。
現状を仕様とするのか、スタンプラリーのタイトルを変えるのか。

@kouho
確認ありがとうございます。
ベルのカウントは参照する変数が間違っていたので、数回余分にカウントしていました。

ベルのカウント箇所は、仕様書通りに実装していますが、もしかしたら認識が違うかもしれないので明日確認します。
!スクリーンショット 2021-06-02 20

@wada
こちら修正箇所含めて確認し、問題ないと思います。

ベルに関してはまた、後に再度確認します。

@kouho @iwamoto
冒険のベルとバナー画像、お知らせIDの設定をしました。
確認をお願いします！！！

@wada
確認遅れてしまってすみません、冒険のベルとバナー画像確認しました。OKです。

お知らせIDについてですが、jrye.master_yamada_campaign.official_info_idの更新をするということでしたらこちら反映されて良さそうでしたので確認お願いいたします。

@kouho
はい、お知らせID更新しました。

@wada
冒険のベルとバナー画像、お知らせIDの設定の確認を行いました
問題なく設定されていると思います。

マージします！！

@wada
お知らせIDの更新も確認しました。OKだと思います。マージ了解です。