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

ティザーサイトの最終調整(リファクタリング含む) #36

Closed
8 tasks done
inouetakuya opened this issue May 1, 2019 · 6 comments · Fixed by #39
Closed
8 tasks done

ティザーサイトの最終調整(リファクタリング含む) #36

inouetakuya opened this issue May 1, 2019 · 6 comments · Fixed by #39

Comments

@inouetakuya
Copy link
Contributor

inouetakuya commented May 1, 2019

この Issue の完了条件

  • ティザーサイト仕様 及び 原稿 に沿うよう修正する
    • Home ページ
    • 行動規範ページ
    • プライバシーポリシーページ
    • お問い合わせページ
  • 沖さんによるレビュー(ティザーサイト公開まで Netlify が使えないので local dev 環境にて)
  • レビューを踏まえて修正
  • kazupon さんによる最終チェック
  • 最終チェックを踏まえて修正

この Issue の対象外

  • Netlify によるティザーサイト公開
@ryamakuchi
Copy link
Collaborator

@inouetakuya
こちら BaseMain の適用 & 調整含めやりたいです。
よろしくお願いします!

@inouetakuya
Copy link
Contributor Author

inouetakuya commented May 2, 2019

@ryamakuchi

ありがとうございます!:raised_hands:

このあたり @ryamakuchi さんつよそうに思っていたでアサインしようかと考えていました。

できるだけプルリクの範囲を小さくしてもらいたいので、やりづらいとかがなければ、下記とは別プルリクエストにしてもらえると有り難いです 🙏

#34 (comment)

code-of-conduct ページはもうすでにマージされているので @ryamakuchi のほうでリファクタリングします

@inouetakuya
Copy link
Contributor Author

共有のためメンションしておきます 😄 > @448jp @kazupon

@ryamakuchi
Copy link
Collaborator

@inouetakuya

できるだけプルリクの範囲を小さくしてもらいたいので、やりづらいとかがなければ、下記とは別プルリクエストにしてもらえると有り難いです 🙏

承知しました!

であれば、先に #34 (comment) のプルリクをやってから全体的な最終調整を行いたいと思います。よろしくお願いします!

@448jp
Copy link
Contributor

448jp commented May 2, 2019

@inouetakuya
了解ですー、今のところGW後半~明けくらいからチェックできるようになる、と考えておけばよさそうですね(認識違っていたらご指摘ください)。

@kazupon
Copy link
Member

kazupon commented May 3, 2019

@inouetakuya
ティザー公開までの条件、問題ないですー

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 a pull request may close this issue.

4 participants