imprv: Behavior when creating first user fails #8801
Conversation
| @@ -27,11 +27,17 @@ module.exports = (crowi: Crowi): Router => { | |||
| const router = express.Router(); | |||
|
|
|||
| // eslint-disable-next-line max-len | |||
| router.post('/', registerRules(), apiV3FormValidator, addActivity, async(req: FormRequest, res: ApiV3Response) => { | |||
| router.post('/', registerRules(), registerValidation, addActivity, async(req: FormRequest, res: ApiV3Response) => { | |||
There was a problem hiding this comment.
apiV3FormValidator middleware ではなく registerRules() に対応する registerValidation middleware を利用するように変更。
| if (!req.form.isValid) { | ||
| const errors = req.form.errors; | ||
| return res.apiv3Err(errors, 400); | ||
| } |
There was a problem hiding this comment.
/register にならって追記。Validation に失敗したらここでエラーを返却する。
growi/apps/app/src/server/routes/login.js
Lines 137 to 140 in d92b768
| ))} | ||
| </p> | ||
| ) | ||
| } |
There was a problem hiding this comment.
LoginForm.tsx にならってサーバーからのエラー結果を form 上部のアラートに表示するようにした
growi/apps/app/src/components/LoginForm/LoginForm.tsx
Lines 356 to 366 in d92b768
|
reg-suit detected visual differences. Check this report, and review them. 🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴 ⚫⚫⚫⚫⚫⚫⚫⚫⚫ What do the circles mean?The number of circles represent the number of changed images.🔴 : Changed items, ⚪ : New items, ⚫ : Deleted items, and 🔵 Passed items How can I change the check status?If reviewers approve this PR, the reg context status will be green automatically. |
yuki-takei
deleted the
imprv/146005-behavior-when-creating-first-user-fails
branch
Task
#146003 [v7] [issue] 最初のユーザーを作成する時に8文字以内のパスワードで登録しようとすると「インストールに失敗しました...」というトースターのみが表示される
┗ #146005 修正
Demo
2024-05-13.16.37.12.mov