search

ProgateHackathon / Gitagram

1 stars 1 forks source link

Feature/#113 remove repository #126

Open am2525nyan opened 2 months ago

am2525nyan commented 2 months ago

やったこと

自分の投稿したリポジトリを消せるようにしました リポジトリを投稿すると新しくdeveloperが作成されるようになってたので、基本的にはアカウント作成時のみ作成されるようにしてます。

影響範囲

developer周り

補足

クリーンアーキテクチャいかせてないのでコメントください。。。。。

スクリーンショット

https://github.com/user-attachments/assets/0f040b6c-0f25-4b06-9878-8b7da8db4083

動作確認

削除確認済み close #113

am2525nyan commented 2 months ago

それに伴って #121 がいらなくなってしまったかもしれない（未確認）

urassh commented 2 months ago

@am2525nyan 投稿したリポジトリ一覧 と 削除機能 でPRを分けて見たいかも！！

urassh commented 2 months ago

@am2525nyan

それに伴って https://github.com/ProgateHackathon/Gitagram/pull/121 がいらなくなってしまったかもしれない（未確認）

なるほど？それはなんでだろう？

am2525nyan commented 2 months ago

あっ、、、、、、、、、、、、、、 コミット分けてないです、、、、、、、、、、、

am2525nyan commented 2 months ago

@urassh DeveloperRepositoryのcreateがpost時に呼ばれなくなったので、そもそもfirestore に同じ名前のdeveloperが現れないからです！

urassh commented 2 months ago

@am2525nyan なるほどね！ 例えば、GitHubでログインする時にGitHubからもらったusernameを毎回createしてなかったっけ？

am2525nyan commented 2 months ago

あーー！忘れてました！！developerID違うと取ってこれないのでログインのとこ修正します！！！！

am2525nyan commented 2 months ago

@urassh 一旦修正してます！

am2525nyan commented 2 months ago

userdefaultってアプリけすと消えましたっけ？

am2525nyan commented 2 months ago

消えますね！！！

am2525nyan commented 2 months ago

@urassh uid使いませんか、、、？？？

urassh commented 2 months ago

@am2525nyan FirebaseAuth使うってなったらアリではあるかなぁ

am2525nyan commented 2 months ago

@urassh
ログインfirebaseAuthなのでありかも！です！

am2525nyan commented 2 months ago

そしたらuserdefault使わなくていいし、developerが〜みたいなの無くなりそうですね！