taisei-13046 / twitter

1 stars 0 forks source link

Re follow #6 #13

Closed taisei-13046 closed 2 years ago

taisei-13046 commented 2 years ago

https://user-images.githubusercontent.com/78260526/141929278-bb84c8d6-8b38-4a8c-bd59-158302384471.mov

taisei-13046 commented 2 years ago

リクエストメソッドごとに分離について

この方針で一度進めましたが、1つのHTML上でFollowとUnFollowをすることができないため、 現状のFollowViewでどちらの操作もするようにしました。 これに関しては,DRFではHTTPメソッドごとに分ける必要性を感じましたが、今回の場合はこのような形でも問題ないかと思いました!ご指摘ありがとうございます!

Follow機能をUserへ移す

これは対応をしました

taisei-13046 commented 2 years ago
スクリーンショット 2021-12-09 19 39 00
taisei-13046 commented 2 years ago
スクリーンショット 2021-12-16 10 27 02

test 実行結果

taisei-13046 commented 2 years ago

方針

今までクラスでFollow, UnFollow機能を作成しようとしていましたが、汎用Viewを使用するとやりずらさを感じたため関数VIewで書く方針に切り替えました その結果、とてもスムーズに書くことができたのですがこの方針はどうでしょうか 関数Viewに切り替えた理由

今後のスタンス

基本はクラスViewで書いていき、細かい処理やtemplateが必要ない処理などは関数Viewで処理する

taisei-13046 commented 2 years ago
スクリーンショット 2021-12-21 0 31 50
taisei-13046 commented 2 years ago
スクリーンショット 2021-12-26 23 40 19