8000 [WIP] WORKSHOPにエリアの項目を追加 by togazo · Pull Request #52 · coderdojo-japan/dojocon2018.coderdojo.jp · GitHub
[go: up one dir, main page]

Skip to content

[WIP] WORKSHOPにエリアの項目を追加 #52

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

[WIP] WORKSHOPにエリアの項目を追加 #52

wants to merge 3 commits into from

Conversation

togazo
Copy link
Collaborator
@togazo togazo commented Aug 13, 2018

どの部屋で行うかを表示できるように、項目を追加しました。

@yasulab
Copy link
Member
yasulab commented Aug 13, 2018

@togazo あ、Session ページと同様の理由で、Sort しやすいように先ほどファイル名に Prefix をつけちゃったので、master から最新のコードを pull しておくと良さそうです...!!

@togazo
Copy link
Collaborator Author
togazo commented Aug 13, 2018

@yasulab 最新のコードをpullしたつもりができてなかったっぽいです。す、すみません…><

@yasulab
Copy link
Member
yasulab commented Aug 13, 2018

無問題 🆗 ✨

@togazo
Copy link
Collaborator Author
togazo commented Aug 13, 2018

@yasulab 絶妙にconflictしちゃったようで、対応の仕方が分からずパニクってます(大汗
2018-08-13 21 28 13

@togazo togazo mentioned this pull request Aug 13, 2018
@yasulab
Copy link
Member
yasulab commented Aug 13, 2018

絶妙にconflictしちゃったようで、対応の仕方が分からずパニクってます(大汗

その場合は1から作りお直すのがラクかもですね ;)

@togazo
Copy link
Collaborator Author
togazo commented Aug 13, 2018

その場合は1から作りお直す

おー、なるほどですー。
初めての体験なので勉強がてらやってみますね!お騒がせしてますー。

@yasulab yasulab changed the title WORKSHOPにエリアの項目を追加 [WIP] WORKSHOPにエリアの項目を追加 Aug 15, 2018
@togazo
Copy link
Collaborator Author
togazo commented Aug 16, 2018

すみません、いま別件立て込んでて放置してますがあとで作業&margeします。ねんのためー

@yasulab
Copy link
Member
yasulab commented Aug 16, 2018

特にWebサイト上で問題になることはないと思うので全然大丈夫です! 😸 👌

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
0