userのget APIにlink追加（作業中

yoko-at-home commented 1 year ago

チケット

closes #166

userのget APIにlink追加作業中ですが、コード参照いただけるようにプルリク投げてます。（Recipeを参考にしたけど、やっている事がわかってない

TopBarのPopoverの呼び出し方と合わせると迷子🥺

yoko-at-home commented 1 year ago

@KohnoTaiyo link 取得できてるっぽいですが、このブランチでどこまで作業すべきでしょうか？続きはまた明日の午後以降

KohnoTaiyo commented 1 year ago

たしかペリおさんがpopbarのSNS周りやっていた気がしますが、このPRで食ってしまってOKです。時間もないのとあちらは進捗もないと思うので

yoko-at-home commented 1 year ago

@KohnoTaiyo 型を修正して、SNSリンク取得ってこんな感じかなーうんうん💦ってやってますが、方向性あってますか？このまま考えていけば良いのか、もっとスマートなやり方があるのかみていただけると助かります！

KohnoTaiyo commented 1 year ago

@yoko-at-home ログインしているユーザーの画面かどうかでの表示出しわけに関しては今手元で自分が実装しているので、それがマージされてから参考にしていただきたいので今回のPRからは外してください！やっていただきたいこととしては

エンドポイントの修正と型に関してはこの調子で進めてしまって問題ないです！！

KohnoTaiyo commented 1 year ago

ちょこちょこコメントしましたが叩いてみて返ってきた値を参考に型を見直してみてください！

yoko-at-home commented 1 year ago

@KohnoTaiyo レビューありがとうとざいます。ちょこちょこと追加しました。 Popbarのソーシャル系アイテムの追加 Userをゲットした時にリンクがレスポンスに含まれる以上2点はクリアしたかなと思います。やろうとしてできないことがTODOとしてコメント入ってます。

yoko-at-home commented 1 year ago

@KohnoTaiyo PopoverItemsの場合わけできちゃったかも〜☆:.｡. o(≧▽≦)o .｡.:☆

yoko-at-home commented 1 year ago

@KohnoTaiyo コードを見れば、そらそやなって思うけど、やっぱり自分じゃ完成できなかった〜😭 ありがとうございます！！