Skip to content

/sotechsha に関連するページを Static にホストするようにしたい #1500

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

Merged
merged 3 commits into from
Feb 14, 2023

Conversation

nanophate
Copy link
Member

@nanophate nanophate commented Feb 12, 2023

やりたいこと

#736 Rails のバージョンを 6.x に上げたいが、Scrivito が 6.x 系に対応していないので、Scrivito でホストしているページを直接アプリ側でホストするように変更して、最終的には Rails 6.x 系に上げたい。

Upgrade Scrivito SDK that supports Rails 6.0 (Currently no support 😭 )
cf. https://www.scrivito.com/rails/release-notes

この PR では /sotechshasotechsha/:pages を対象にStaticページを用意しました。

やらないと

/sotechsha2 の対応

確認方法

https://coderdojo.jp/sotechsha のページと localhost:3000/sotechsha のページを比較して問題がないか確認してほしいです。

Copy link
Contributor

@rakuda-san-desu rakuda-san-desu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

http://localhost:3000/sotechsha以下、それぞれを確認しました!良さそうです✨

@yasulab
Copy link
Member

yasulab commented Feb 13, 2023

@nanophate こちらの PR では https://coderdojo.jp/sotechsha にのみ対応していて、後で https://coderdojo.jp/sotechsha2 に対応するって認識で大丈夫でしょうか? 👀 💭 (「この PR でやらないこと」というセクションを設けると良いかなと思いました!)

@nanophate
Copy link
Member Author

@yasulab はいその認識してで問題ないです。 sotechsha2 の対応はこの PR ではしません。

Copy link
Contributor

@Yuppymam Yuppymam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

対応ありがとうございます!!✨

@nanophate
Copy link
Member Author

こちらマージします!何かあった場合は下記の Rollback を準備しています。

v2789  Deploy ae045cb6   2023/02/14 20:01:32 +0900 (~ 1h ago)
v2788  Deploy e18548ef   2023/02/14 18:46:37 +0900 (~ 3h ago)

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.

4 participants