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.

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

思ったことメモ #14

Open
h-takesg opened this issue Mar 4, 2023 · 19 comments
Open

思ったことメモ #14

h-takesg opened this issue Mar 4, 2023 · 19 comments

Comments

@h-takesg
Copy link
Contributor

h-takesg commented Mar 4, 2023

趣旨

後で相談したほうがいいと思ったことを書いておく
修正するかどうか,いつやるかは知らない
学生部屋でやってる重箱の隅つつき隊のノリ

個別にissueを立てるべきな気もするが粒度が小さそうだったり,個人の好みだったり,しょうもなかったりするのでまとめる

修正するときにIssue化かPull request化

@h-takesg
Copy link
Contributor Author

h-takesg commented Mar 4, 2023

apiのuriが動詞を含んでいる
http://localhost/backend/member/add

uriは名詞として
POST とか PUT メソッドで処理を切り替えたい

#39 で突っ込んだ

@h-takesg
Copy link
Contributor Author

h-takesg commented Mar 4, 2023

backendの user/add と item/add で挙動が違う

重複エントリーに対して
user: failed
item: success
が返ってくる

#39 で揃えた

@h-takesg
Copy link
Contributor Author

h-takesg commented Mar 4, 2023

item addのリクエストボディのプロパティgroupinggroupでいいのでは
#39 で変えた

@h-takesg
Copy link
Contributor Author

h-takesg commented Mar 4, 2023

./init_database.sh でbackend経由で初期データを打ち込むよりは
dbコンテナ立ち上げ時にinit用スクリプト読ませるほうが役割として自然

backendのテストならそう切り分けるべき

@h-takesg
Copy link
Contributor Author

h-takesg commented Mar 4, 2023

teature -> teacher

#33

@h-takesg
Copy link
Contributor Author

h-takesg commented Mar 4, 2023

ログイン時のアラートは好みじゃない

ログイン成功はログイン表示への遷移で示したい
失敗時もパスワード欄付近にメッセージを出力するぐらいがいい

成功にせよ失敗にせよ次のアクションがスムーズになる

どれかで消えた

@h-takesg
Copy link
Contributor Author

h-takesg commented Mar 4, 2023

ホームタブの余白調整していきたい
#41 でいい感じになった

@h-takesg
Copy link
Contributor Author

h-takesg commented Mar 4, 2023

Active/In active -> Visible or Active

可視性だけならvisible
購入不能など有効無効ならactive

トグルスイッチの欄なので有効時のラベルさえあればいいと思う

@h-takesg
Copy link
Contributor Author

h-takesg commented Mar 4, 2023

ユーザーや商品のソート/フィルターがあるべき

@h-takesg
Copy link
Contributor Author

h-takesg commented Mar 4, 2023

今月の支払い分 -> 今月の購入分 or 今月

@h-takesg
Copy link
Contributor Author

h-takesg commented Mar 4, 2023

docker-compose.ymlが分離したの辛い
Dockerfile PreProcessorみたいなのほしい

変数で解決した

@h-takesg
Copy link
Contributor Author

h-takesg commented Mar 5, 2023

ユーザー/商品削除前にアラートが欲しい

@h-takesg
Copy link
Contributor Author

h-takesg commented Mar 14, 2023

backend/bin/ ディレクトリがignoreされていない

#21 で突っ込んだ

@h-takesg
Copy link
Contributor Author

h-takesg commented Mar 21, 2023

frontend側でbackendと通信する処理をまとめたい
Backendクラスみたいな

#21 で突っ込んだ

@h-takesg
Copy link
Contributor Author

h-takesg commented Mar 24, 2023

backendまわり,idっぽいものをnameとしている
直した

@h-takesg
Copy link
Contributor Author

h-takesg commented Mar 26, 2023

setSumPurchased必要?
#66 で消えそう

@h-takesg
Copy link
Contributor Author

h-takesg commented Apr 3, 2023

heic投げられたらどうなる?
frontがファイル選択時点で弾くのでまともに使っている間は考えなくていい
backは弾いてないのでそのままdbに格納しそう

@h-takesg
Copy link
Contributor Author

h-takesg commented Apr 4, 2023

商品を削除すると購入履歴はどうなる?

@h-takesg
Copy link
Contributor Author

追加時点でactiveがfalseなのはなぜ?
追加した=今から使うもの では?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant