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

Issue16 #49

Merged
merged 16 commits into from
May 24, 2015
Merged

Issue16 #49

merged 16 commits into from
May 24, 2015

Conversation

d-yano
Copy link
Member

@d-yano d-yano commented May 24, 2015

No description provided.

@d-yano
Copy link
Member Author

d-yano commented May 24, 2015

#16 閉じました。

@d-yano
Copy link
Member Author

d-yano commented May 24, 2015

ダイアグループの重複登録が可能ですが、増便ダイアなどで複数のダイアグループを1日に割り当てる事が可能でもいいと思います。
どうでしょうか?
#53

d-yano added a commit that referenced this pull request May 24, 2015
@d-yano d-yano merged commit a205f88 into master May 24, 2015
@mugifly
Copy link
Member

mugifly commented May 24, 2015

おっと...見ている瞬間にマージされたのですが。

@mugifly
Copy link
Member

mugifly commented May 24, 2015

ダイアグループの重複登録が可能ですが、増便ダイアなどで複数のダイアグループを1日に割り当てる事が可能でもいいと思います。
どうでしょうか?

例えば特定の1日において、通常ダイアと特別ダイアを結合して取り出せる感じでしょうか?
それでも良いかもしれませんが、単純に通常ダイアをコピーして特別ダイアを書くような簡素な設計でも良いとは思います。もしよければ、別途議論しませんか?

#53

@d-yano
Copy link
Member Author

d-yano commented May 24, 2015

例えば特定の1日において、通常ダイアと特別ダイアを結合して取り出せる感じでしょうか?

はい

ダイアのコピー機能 #36 コピーは、ダイア追加時の機能として追加したいと考えています。
#53

おっと...見ている瞬間にマージされたのですが。

すみません、日曜だったもので。

@d-yano
Copy link
Member Author

d-yano commented May 24, 2015

もしよければ、別途議論しませんか?

そうですね、仕様としてきちんと決めたほうがいいですね。

#53 つくりました。

@mugifly
Copy link
Member

mugifly commented May 24, 2015

すみません、日曜だったもので。

たしかに24時間以内にレビューできていないのは私の責任かもしれませんが、土曜日が研修だったのも伝えていましたし、休日はPCを見ないときもあるので考慮はしてほしいです。休日だからといって自分でマージしてたらあまり意味が無いのでは?という意味も含めて。

まぁWebサービスはおかしければRevertすれば済む話というのも確かですので、目くじら立てるほどでも無いですけど(笑)
何はともあれお疲れ様です。

@d-yano
Copy link
Member Author

d-yano commented May 25, 2015

申し訳ありません

ありがとうございす.

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