Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Goonの2nd gen移行 #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Goonの2nd gen移行 #9

wants to merge 1 commit into from

Conversation

Rogietp
Copy link

@Rogietp Rogietp commented Nov 22, 2022

目的・概要

  • 本家Goonは2nd gen対応済のバージョンをリリースに伴い、2nd gen対応させる。

(必要に応じて)動作確認

  • GAE gen2にて全ての機能が正常に動作する

@Rogietp Rogietp self-assigned this Nov 22, 2022
@yns-kume
Copy link

#8 (review) にて共有いただいておりましたフィードバックを含め新たにPR作成させていただきましたのでご確認いただけますと幸いです。

本家における以下赤枠の修正に関しましては、以前のバージョンv1にてtopgateのレポジトリ用にカスタマイズされていた箇所があるため、反映しない状態にしております。
image

カスタマイズしている例 ;
github.com/mjibson/goon/v2にてMemcacheExpirationという変数が存在しない
image

こちら、カスタマイズされている箇所のgithub.com/mjibson/goon/v2への適応を含めて対応が必要な場合には追って作業を実施させていただきます。

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.

2 participants