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

【ver2.0.0-prototype】リファクタリング #24

Open

smctrl opened this issue Mar 26, 2022 · 0 comments

Assignees

Member

smctrl commented Mar 26, 2022

フロント

コンポーネントでfetchしてるやつはserviceに切り出す
st-user-managerのroleOptionsを定数用ファイルに切り出す
st-user-managerのsaveメソッドのroleChange.roleの分岐でelseをelse ifに修正する
authentication-serviceの異常系を考える（おそらく単純にリトライでOK）
st-search.htmlで特殊確保力や特殊コストの判定を無理やりやっている箇所を修正する

バック

実行オプションを設定ファイルから拾うように変更する
コメント少ないから書くこと
自分用に作ったビルド・実行・停止用のスクリプトを共有する
README.mdでPython実行のくだりがあるが、そこはcrontabに書いたやつを説明に流用する
saconfig.jsonを読む際にファイルの位置をコマンドライン引数で受けるようにする
saconfig.jsonのパースエラーをハンドルする
アカウントの一時登録データをバリデータで使いたいのでqueue系をサービスに切り出す
排他制御が必要な箇所の洗い出し（大変なのでユーザー少ないうちは優先度低）
console.logをlogger利用に修正する

smctrl self-assigned this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment