PR

CORSエラー対応 #254

wada opened this at 2020-09-14T19:42:55; closed

Description

opened by wada at 2020-09-14T19:42:55; updated 2023-02-15T14:00:13

issue #250

一旦プルリク作成してます
現在、冒険ページのみ対応しているので、他の主要なページの対応が完了したらレビューを依頼します。

wada change_title at 2020-09-14T19:43:10: [WIP]CORS対応 [WIP]CORSエラー対応
wada add_label at 2020-09-15T21:41:57: レビュー待ち
wada change_title at 2020-09-15T21:42:02: [WIP]CORSエラー対応 CORSエラー対応

Comment

#7119 by wada at 2020-09-15T21:45:17; updated 2020-09-15T21:45:17

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

ajaxでtoken発行しようとしたらエラーコードを返すようにしました。
ajax戻り値のエラーコードが指定値の場合、TOPへ遷移させます。
メインページと冒険は、ajaxの戻り値(json)が空だと動作しないので、jsonがnullでもTOPへ遷移するように修正しました。

developマージ後に本番jsとキャッシュ対応します。

Comment

#7120 by iwamoto at 2020-09-15T23:06:10; updated 2020-09-15T23:06:10

@wada
こちら動作も確認しました、問題ないと思います。

iwamoto delete_label at 2020-09-15T23:08:25: レビュー待ち
wada merge at 2020-09-16T09:57:57: CORSエラー対応
wada close at 2020-09-16T09:57:57: Close

Commit range

base84314bebe0c951cad9bf385f5dd0bfa12fe97eca
head00563e9f3ca1b6f0d750cbfc859a417e5dcffde8
base branchjolly/jr-web:develop
head branchjolly/jr-web:fix_cors_250

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed4 filesView patch
statusfile
modifiedlib/SubFunc/SubFunc_yamada.inc
modifiedlib/conf/define.inc
modifiedpublic_html/js/JOLLYROGER.js
modifiedpublic_html/js/JOLLYROGER_mission.js
Patch body is stored as a separate file to keep this page lightweight.