Skip to content

Update index content #25

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
Dec 6, 2016
Merged

Update index content #25

merged 3 commits into from
Dec 6, 2016

Conversation

nanophate
Copy link
Member

@nanophate nanophate commented Dec 3, 2016

やりたいこと

CoderDojoのサイトを丸々表示できるようにする。

やること

  • 使わないwidgetの機能を消去して、customWidgetを作成
  • HTMLとCSSのデータをcoderdojo.jpからコピペで移行
  • コピペしたデータを修正
    • コンテツが表示される
    • デザインが一致している
    • レスポンシブである
      • 文字の表示
      • モバイル対応

@nanophate nanophate force-pushed the update_index_content branch 2 times, most recently from 666a194 to b38deae Compare December 3, 2016 18:48
@nanophate nanophate force-pushed the update_index_content branch from b38deae to fd7e3fb Compare December 4, 2016 09:04
@yasulab
Copy link
Member

yasulab commented Dec 6, 2016

良さそう! 😸 ✨ とりあえずマージしちゃいますね ;)

あといつも言っているけど、今回もテストがないので、まずは「タイトルやフッターに "CoderDojo Japan" があるかどうか」みたいな簡単テストからでもいいので、テスト書きましょう💦

@yasulab yasulab merged commit ce5590a into master Dec 6, 2016
@yasulab yasulab deleted the update_index_content branch December 6, 2016 02:10
@yasulab
Copy link
Member

yasulab commented Dec 6, 2016

@nanophate とはいえ、トップページが復元できたので、12/11 (日曜) のドメイン切り替え ( #23 (comment) ) には余裕を持って間に合いそうで良かったです! 👍 ✨

あとはテストとインフラ周り (Heroku, Circle CI, ドメイン設定など) ですかね ;)
インフラ周りは僕も手伝えるので、まずはテストの方をお願いします! ✅ 🔧 💨

@nanophate
Copy link
Member Author

nanophate commented Dec 6, 2016

@yasulab

トップページが復元

まだ全国のCodorDojo の読み込みができていません。
今日中には対応したいと思います。
そのまま現在あるウェブサイトからそのまま表示するようにします。

来週中には直接データベースから取得できるようにします。

@nanophate
Copy link
Member Author

#26 で各dojoを静的に表示できるようにしました。
動的には来週実装しようと思います。

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