Skip to content
This repository was archived by the owner on Mar 19, 2020. It is now read-only.

#2 Add Beta alert modal. #42

Merged
merged 2 commits into from
Sep 6, 2018
Merged

#2 Add Beta alert modal. #42

merged 2 commits into from
Sep 6, 2018

Conversation

yum45f
Copy link

@yum45f yum45f commented Sep 6, 2018

fixed: #2

  • ページを開いた際、モダールウィンドウが開いて開発中の旨を伝えてくれるよう実装

    • アクセス履歴を Cookie に保存するため、その後ページの遷移のたびに表示しない
    • Cookie は 7日語に消滅する
  • モーダルウィンドウに 6時間の睡眠につての説明も記載

@yum45f yum45f added the Improvement 改善事項 label Sep 6, 2018
@yum45f
Copy link
Author

yum45f commented Sep 6, 2018

CI通過後にマージ

@yum45f
Copy link
Author

yum45f commented Sep 6, 2018

js.cokkie.js のコードスメルを指摘されてるw

@yum45f
Copy link
Author

yum45f commented Sep 6, 2018

Vendor に移動させとこうw

@yum45f
Copy link
Author

yum45f commented Sep 6, 2018

Merge! ✅

@yum45f yum45f merged commit b10cc82 into master Sep 6, 2018
@yum45f yum45f deleted the add/beta-alert branch September 6, 2018 16:17
@yum45f yum45f restored the add/beta-alert branch September 6, 2018 16:56
@yum45f
Copy link
Author

yum45f commented Sep 6, 2018

JS ライブラリの参照をど忘れしてた。

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Improvement 改善事項
Projects
None yet
Development

Successfully merging this pull request may close these issues.

「このサイトはまだ開発中です」的な感じのポップアップ表示したい。
1 participant