Project

General

Profile

Feature #733

Friends

Added by yusuke kokubo about 8 years ago. Updated about 8 years ago.

Status:
終了(Closed)
Priority:
通常(Normal)
Assignee:
Target version:
Start date:
05/10/2011
Due date:
% Done:

0%

Estimated time:

Description

やりたいこと

タイムラインのときにちょろっと話がでましたが

  • 自分がスキルに賛同した人
  • 自分のスキルに賛同してくれた人

を友達関係としてつくりたいと思ってます。

狙い

現状では「あの人にスキルをつけよう!」と思ったときにまずそのユーザーを探す手間をかけるところからはじめないといけません。
親しい人とのスキルを追加したり賛同したりをもっと身近にするためにFriendShipモデルを作って、関係をつくったらどうかな、と思いました。


Files

Associated revisions

Revision 0e69f12c (diff)
Added by yusuke kokubo about 8 years ago

follow/follower/friends関係を作成 refs #733

Revision 3be73fae (diff)
Added by yusuke kokubo about 8 years ago

following/followerをKeyからEmailに変更した refs #733

Revision 79e96920 (diff)
Added by yusuke kokubo about 8 years ago

followerとfollowingの突き合わせをちゃんとした refs #733

Revision 5c5a683b (diff)
Added by yusuke kokubo about 8 years ago

単純にFriendsを羅列するだけのUI refs #733

Revision b4c4020b (diff)
Added by yusuke kokubo about 8 years ago

既存データがあったり、例外が起きた場合にmigrationできなかったのを修正
refs #733

Revision 3f768cdf (diff)
Added by 啓介 大橋 about 8 years ago

feture733 refs #733 Frends

History

#1

Updated by 啓介 大橋 about 8 years ago

タイムラインとひもづけないならEntityとしてはありですね

あった方がいい機能なんで作りましょう!

#2

Updated by yusuke kokubo about 8 years ago

賛同ありがとうございます。

対応としては、Home | MyPage | Users | Skills
にFriendを加える方向でどうでしょう。

とりあえずは、ログインユーザーとFriend関係のあるユーザーを羅列する感じで。

#3

Updated by 啓介 大橋 about 8 years ago

何となくですがMyPageは右上のログインアカウント名のところにリンクでもいいかもですね

あまりルートナビゲーションが多いのは好きじゃないっす、、、
Frendsも追加でもいいですがuserにいれてしまってもいいかもですね

#4

Updated by yusuke kokubo about 8 years ago

MyPageは確かにそうですね。

FriendsとUsersを一緒にすると中でゴチャゴチャしちゃうような感じがするので、
とりあえずわけてちょっと様子を見たいところです。

#5

Updated by 啓介 大橋 about 8 years ago

なるほど!
確かにユーザというところだけみるとまとめた方がいいですが
これから昨日が追加されることを考えると分けといた方が良さそうですね

MyPageに関してはありがとうございます!

#6

Updated by yusuke kokubo about 8 years ago

  • Subject changed from Friend to Friends

Friendsだと友達なので、相互フォローしてることを言うので、
A -> B にスキルがついてても
B -> A がなければFriendとは呼べないかなぁ。

やっぱりFollow/Follwerの関係をつくって、
相互FollowをFriendとするモデルが良いのかも。

#7

Updated by 啓介 大橋 about 8 years ago

yusuke kokubo は書きました:

Friendsだと友達なので、相互フォローしてることを言うので、
A -> B にスキルがついてても
B -> A がなければFriendとは呼べないかなぁ。

やっぱりFollow/Follwerの関係をつくって、
相互FollowをFriendとするモデルが良いのかも。

そうですね。
ちょっとモデルが難しそうですけど。。。

#8

Updated by yusuke kokubo about 8 years ago

ちょっとモデルとRPCを作ってみたので見てもらってよいですか。
まだ全然動作確認もしてないのですがこんな感じでやろうかと。

#9

Updated by 啓介 大橋 about 8 years ago

FollowingとFollowerの型がKeyになっていますが、
e-mailにしておくと、getUsersByEmailでMemcacheから取れると思うのですがどうでしょう??
※Key、Email両方もって置くもありですが

それ以外はモデル的に問題ないと思います!!

#10

Updated by yusuke kokubo about 8 years ago

e-mailに変更しました。

githubにfriendsというブランチでpushしたので確認してください。
https://github.com/YusukeKokubo/SkillMaps/tree/friends

UI側はお任せしちゃってもよいですか?

#11

Updated by 啓介 大橋 about 8 years ago

  • Assignee changed from yusuke kokubo to 啓介 大橋

了解です!
いつも通りカクーでつくって共有します!

#12

Updated by yusuke kokubo about 8 years ago

まずはUsersみたいにだらだらと一列に並べるだけで良いと思ってます。
その後に必要な機能はまた思いつくでしょう。

#13

Updated by 啓介 大橋 about 8 years ago

そうすると一旦UIはUserListUIを
利用して、Presenterだけ違うのにしてもいいかもですね

UserListUIに機能が増えればFriendsも増えるような

異なる機能が出たら継承して拡張か、ラップして委譲する感じで

いったんはままでいいと思いますがis aな関係でもいい気がします

#14

Updated by yusuke kokubo about 8 years ago

啓介 大橋 は書きました:

そうすると一旦UIはUserListUIを
利用して、Presenterだけ違うのにしてもいいかもですね

よいと思います!

#15

Updated by 啓介 大橋 about 8 years ago

了解です!
では #736 と並行して実施します!

#16

Updated by yusuke kokubo about 8 years ago

ちょっとこちらで試してます。

#17

Updated by 啓介 大橋 about 8 years ago

もうしわけないです、、、

#18

Updated by yusuke kokubo about 8 years ago

既存のUserListを見よう見まねでつくってみました。

http://feature-733.skillmaps.appspot.com/

# MVP難しい…

#19

Updated by 啓介 大橋 about 8 years ago

おお!

MVPに関しては観点とか含めて
かるーく資料があった方がいいですよね

近いうち(6月)に会社向けの資料を作るので
展開しますねー

#20

Updated by yusuke kokubo about 8 years ago

本当はログインしてなかったらmyPageみたいにログインをうながす仕掛けをつくったりしたいのですが
そこはお願いしても良いですか?

すみません、他人任せで…

#21

Updated by 啓介 大橋 about 8 years ago

了解です!
むしろ最近コントリビュートできてないので
やらせてください!!

#22

Updated by 啓介 大橋 about 8 years ago

ちょろっと思ったことベースで書くのですが、
非ログイン時はmyPageとfrendsってheaderにも出さない方がいいですか??

#23

Updated by yusuke kokubo about 8 years ago

非ログイン時はmyPageとfrendsってheaderにも出さない方がいいですか??

その方が良いかもしれませんね...!!

#24

Updated by 啓介 大橋 about 8 years ago

了解です!

Frendsの機能自体には直接アクセスされたときのために
促すしかけを入れておきますがheaderからも消えるようにしておきます(`・ω・´)キリッ

#25

Updated by yusuke kokubo about 8 years ago

お願いしますm(_ _)m

#26

Updated by yusuke kokubo about 8 years ago

そういえば、Friendsは両思いしかないんですけど、follow/folllowerも表示するビューがどこかに欲しいですね。

#27

Updated by 啓介 大橋 about 8 years ago

了解です。
タブかアンカーで切り替えられるようにしておきます。

#28

Updated by yusuke kokubo about 8 years ago

  • Status changed from 担当(Assigned) to 終了(Closed)

とりあえず目的の機能は達成できたと思います。

その他いろいろ思いつくことがあればまたそのうち。

Also available in: Atom PDF