Skip to content

Feature/migration#210

Merged
kt3k merged 49 commits intonodejsjp:masterfrom
kt3k:feature/migration
Nov 13, 2016
Merged

Feature/migration#210
kt3k merged 49 commits intonodejsjp:masterfrom
kt3k:feature/migration

Conversation

@kt3k
Copy link
Copy Markdown
Member

@kt3k kt3k commented Nov 12, 2016

https://iojs-jp.slack.com/messages/nodejsjp/ の方で議論しているリニューアルの PR です。

デモサイト
https://kt3k.github.io/asdfghjs.jp/

マージ前作業:

  • CircleCI のビルドを on に
  • CircleCI Publish 用の Deploy Key をリポジトリに設定

マージ後作業:

  • master を source ブランチに改名する (手順1)
  • CircleCI で source ブランチをビルドして、master に push される (のを待つ) (手順2)

Note: この PR にマージ後 上 (手順2) 完了までは、一瞬サイトが表示でき無い状態になります


bulbofile.js でアセットのビルド方法を指定して、開発時は npm run serve で確認可。build 時は同内容が、npm run build でビルドされる、という構成です。(詳しくは README 参照)

Comment thread README.md
url: https://www.google.com/
---

当社は1945年設立、2011年マザーズ上場の若い企業で、街コンワールド( http://machicon-world.com/ )という街コンポータルサイトを運営しています。
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ドメインが失効されているっぽい気がします(?)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

あー、すみません、求人情報データは架空データなので、消さ無いとダメでした。消します。
指摘ありがとうございます。 🙇

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

ダミー求人データを消しました。デモサイトの方で確認できます。 https://kt3k.github.io/asdfghjs.jp/jobboard.html

この行自体は README の中のサンプルなので、そのままにしておきます。

Comment thread old/index.html Outdated
<link type="text/css" rel="stylesheet" href="style.css" />
<link type="text/css" rel="stylesheet" href="http://nodejs.jp/nodejs.org_ja/sh_vim-dark.css" />
<script src="http://nodejs.jp/nodejs.org_ja/sh_main.js"></script>
<script src="http://nodejs.jp/nodejs.org_ja/sh_javascript.min.js"></script>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

問題なければscriptタグはbodyの最後にしたほうがいいと思います:)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

動作確認したところ、後ろに下げても問題なさそうだったので、下げます。指摘ありがとうございます!

@kt3k kt3k merged commit 5b8ea4f into nodejsjp:master Nov 13, 2016
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