Skip to content

chore: gas/rescue/ をライブ(重複防止を入れたバージョン4)に合わせる - #550

Merged
taminororo merged 2 commits into
developfrom
chore/kanba/549/sync-live-rescue-gas
Sep 19, 2026
Merged

taminororo merged 2 commits into
developfrom
chore/kanba/549/sync-live-rescue-gas

Conversation

@taminororo

Copy link
Copy Markdown
Collaborator

対応Issue

resolve #549

概要

gas/rescue/ を、9/19 に重複防止を入れたライブ(ウェブアプリのデプロイのバージョン4)に合わせる。

  • 1つ目のコミット:ライブの コード.js を取り込むだけ(修正は含めない)。onChange.js と appsscript.json はライブと一致していた
  • 2つ目のコミット:gas/README.md の対応表で「未確認」のままだった rescue/ の行を更新

取り込んだ変更(ライブには反映済み)は次のとおり。

doPost: 「種類・学籍番号・内容」から作ったキーを CacheService に180秒だけ記録する。
        同じキーが来たらスプシに追記せず、API へ PUT /{type}-rescues/{重複の対応番号} で
        status=done と「対応番号Xにまとめました」を書く(送った人のアプリに返答の来ない行を残さない)。
        CacheService が使えないときは追記する側に倒す(重複より取りこぼしのほうが害が大きい)。

背景は #547 / #548。GAS の応答の2段階目が壊れて API が書き込み済みのレスキューを失敗扱いにし、委員が押し直してスプシに同じ行が並んでいた。API 側の根本対応は #548、この重複防止はその応急処置で、#548 の反映後も押し直しへの保険として残す。

テスト項目

  • ライブ(HEAD とバージョン4)の3ファイルと、このブランチの gas/rescue/ が一致すること(cmp で確認済み)
  • 全 .js を連結して node --check が通り、トップレベルの名前の重複がないこと(確認済み)
  • 反映前に、GAS のサービスをモックした Node のテストで12項目を確認済み(押し直しは追記しない、内容・送信者・人数が違えば追記、キャッシュが使えなくても追記、電話番号の先頭0を保つ処理が従来どおり、など)。テストは gas/ に置いていない(clasp push で GAS に送られてしまうため)

備考

ライブへの反映は 9/19 15:42 JST に済んでいる。戻すときは同じデプロイIDをバージョン3に向け直す(clasp deploy -i <デプロイID> -V 3)。

9/19 にライブへ入れた重複防止(同じ人・同じ種類・同じ内容を180秒以内に受けたらスプシに追記せず、DBの重複行を対応済みにしてまとめ先を書く)を取り込む。取り込みのみで修正は含めない。onChange.js と appsscript.json はライブと一致していた。
最終同期が「未確認」のままだったので、9/19 にライブと全3ファイル一致を確認したことを書く。ウェブアプリはバージョン固定で、push だけでは本番に反映されないことも添える。
@coderabbitai

coderabbitai Bot commented Sep 19, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 10 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository: NUTFes/SeeFT/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 653fd5fd-18ca-4ed2-8647-4601c1667d13

📥 Commits

Reviewing files that changed from the base of the PR and between 99f0830 and 5d69ba4.

📒 Files selected for processing (2)
  • gas/README.md
  • gas/rescue/コード.js

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@taminororo
taminororo merged commit e73e9ed into develop Sep 19, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

gas/rescue/ をライブ(重複防止を入れたデプロイのバージョン4)に合わせる

1 participant