PR

SWF告知の画像を再利用 #2552

wada opened this at 2023-01-20T15:29:14; closed

Description

opened by wada at 2023-01-20T15:29:14; updated 2023-01-26T14:55:54

概要

SWF告知の画像を再利用 #2550

カラム追加SQL

ALTER TABLE official_info_event ADD `reuse_image_id` int(11) DEFAULT NULL COMMENT '再使用したい画像の告知ID';

TODO

  • [x] カラム追加
  • [x] webデプロイ
  • [x] 管理画面更新

Comment

#25196 by wada at 2023-01-20T15:29:26; updated 2023-01-20T15:29:26

@kouho
確認お願いします。

kouho add_label at 2023-01-20T15:34:21: 皇甫 LGTM

Comment

#25198 by kouho at 2023-01-20T15:34:30; updated 2023-01-20T15:34:30

@wada
確認OKです

wada merge at 2023-01-20T17:22:02: SWF告知の画像を再利用
wada close at 2023-01-20T17:22:02: Close
wada delete_branch at 2023-01-20T17:22:07: reuse_info_event_image

Comment

#25304 by kouho at 2023-01-26T13:56:15; updated 2023-01-26T13:56:15

@wada
こちらの対応レビュー時点では問題ないと思ったのですが、
1枚目のswfと2枚目のswf画像分けて再利用できる方が融通が効きそうだと思いました。

例えばイベントやキャンペーンの1枚目を再利用しても2枚目は再利用しないというケースも多いですので。

Comment

#25306 by wada at 2023-01-26T14:55:54; updated 2023-01-26T14:55:54

@kouho
そうですね。その対応あると良いですね。

今回は、SWF画像追加作業を0にできるような対応を考えていたので、一括で設定するようにしました。
管理画面の調整もそちらの方が簡単だったので。

Commit range

base65a91c4d8781cdb7ac4eb46a3bcf9b78df10b5a0
head8edeaeb09f7eccdd5a6da3bd55a5c28bcae4b02f
base branchjolly/jr-web:master
head branchjolly/jr-web:reuse_info_event_image

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed2 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/public/Main.inc
Patch body is stored as a separate file to keep this page lightweight.