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

[foofleDrive] リファクタリング #4

Open
Alpha9n opened this issue Oct 27, 2024 · 4 comments · May be fixed by #5
Open

[foofleDrive] リファクタリング #4

Alpha9n opened this issue Oct 27, 2024 · 4 comments · May be fixed by #5
Assignees
@gobotan

Comments

@Alpha9n
Copy link

Alpha9n commented Oct 27, 2024 •
edited
Loading

No description provided.

@Alpha9n Alpha9n linked a pull request Oct 29, 2024 that will close this issue
Open
@gobotan
Copy link
Member

gobotan commented Oct 30, 2024

ソースコードでは定期請求は発生しうる。
実際に本番環境では定期請求部分のロジックが削除されていたため、定期請求が発生しないと考えられる。

@Alpha9n Alpha9n linked a pull request Oct 30, 2024 that will close this issue
【リファクタリング】現状のソースコードをよりわかりやすい形にメソッド分割を実施 #5
Open
@Alpha9n
Copy link
Author

Alpha9n commented Oct 31, 2024 •
edited
Loading

ソースコードでは定期請求は発生しうる。 実際に本番環境では定期請求部分のロジックが削除されていたため、定期請求が発生しないと考えられる。

ここを本番環境へのデプロイまでのissueとして大丈夫ですか？

@gobotan
Copy link
Member

gobotan commented Oct 31, 2024

現状のコードを本番環境で動かすとパフォーマンスが悪く致命的な問題が起きる可能性は大いにある。
ただし、issue自体は解決しているため対応は任せる。

@Alpha9n
Copy link
Author

Alpha9n commented Oct 31, 2024

承知

@Alpha9n Alpha9n changed the title [foofleDrive] 定期請求を復活させる [foofleDrive] リファクタリング Nov 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gobotan gobotan

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@gobotan @Alpha9n