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.

Sign up for GitHub

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

Jump to bottom

[wip] Backbone.Marionette を使ってみる #88

Merged
merged 5 commits into from
Apr 17, 2014
Merged

[wip] Backbone.Marionette を使ってみる #88

merged 5 commits into from
Apr 17, 2014

Conversation

takuyan
Copy link
Collaborator

@takuyan takuyan commented Apr 13, 2014

No description provided.

@takuyan
Copy link
Collaborator Author

takuyan commented Apr 13, 2014

これ、Backbone.Marionette使ってみようと思っているけど、
AngularとかEmberとか色々なバージョンを見てみたい欲があるね。

@tbaba
Copy link

tbaba commented Apr 13, 2014

backbone.ktra.com, angular.ktra.com, ember.ktra.com
と言うことでしょうか！！

@taea
Copy link
Owner

taea commented Apr 13, 2014

色々やるといいと思うｗ

@takuyan
Copy link
Collaborator Author

takuyan commented Apr 13, 2014

なにそれカッコいい( ˘ω˘)"

@ppworks
Copy link
Collaborator

ppworks commented Apr 14, 2014

UsersControllerではない理由は
REST的にはリソース名がmodelに引っ張られる必要がないという背景があり
Userモデルに限らずアカウント情報的な何かをJSONでコネコネ出来るように
という意図を汲み取りました。

あってますか？

@takuyan
Copy link
Collaborator Author

takuyan commented Apr 14, 2014

「うぐっ！」となったけどわたしの暗黙的なアレを明文化すると、その通りだと思います！#本当なのか
あと、この設計にしておくとAccounts::ProfilesControllerでPUT account/profileとか、Accounts::EmailsControllerでPUT account/emailで更新っていうのが自然にできるので、これにしてます！

@takuyan
Copy link
Collaborator Author

takuyan commented Apr 16, 2014

ここまでのコミットの内容、EmberでもAngularでも使いそうなので、一度本体にマージしてはどうだろうかという提案。

@ppworks
Copy link
Collaborator

ppworks commented Apr 16, 2014

API整備みたいな名目でマージしよう

@takuyan
Copy link
Collaborator Author

takuyan commented Apr 16, 2014

マージ用のアレをこっちに用意した #91

@taea taea merged commit 7c29738 into master Apr 17, 2014
@takuyan
Copy link
Collaborator Author

takuyan commented Apr 17, 2014

アレ？こっちもマージされたのか( ˘ω˘)ｽﾔｧ

@ppworks
Copy link
Collaborator

ppworks commented Apr 17, 2014

revisionが同じだから、勝手にcloseされてしまうのかもしれない

@takuyan
Copy link
Collaborator Author

takuyan commented Apr 17, 2014

なるほど！
とりあえず新しいwipプルリだします！

@takuyan
Copy link
Collaborator Author

takuyan commented Apr 17, 2014

続きはウェブで => #92

@taea
Copy link
Owner

taea commented Apr 17, 2014

ミサワ

@takuyan
Copy link
Collaborator Author

takuyan commented Apr 17, 2014

ｗｗｗ

@takuyan takuyan deleted the make_backbone_marionette branch April 17, 2014 03:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@takuyan @tbaba @taea @ppworks