Skip to content

feat(FI-06): implement UI register, login, forgot password and reset …#7

Open
VANDUY-1910 wants to merge 2 commits intodevfrom
feat/FI-06
Open

feat(FI-06): implement UI register, login, forgot password and reset …#7
VANDUY-1910 wants to merge 2 commits intodevfrom
feat/FI-06

Conversation

@VANDUY-1910
Copy link
Copy Markdown
Collaborator

No description provided.

@NHSon05
Copy link
Copy Markdown
Collaborator

NHSon05 commented Apr 27, 2026

register.zod.ts

  • password and confirmPassword validation yêu cầu độ dài tối thiểu 5 ký tự khi nhập sai.
  • password và confirmPassword chưa có bước kiểm tra khớp mật khẩu trước ghi gửi đi.

reset-password-zod.ts

  • Min Length của PW khi nhập form yêu cầu là 5. Còn Min Length của PW khi yêu cầu reset là 8???
  • Ban đầu: Khi khởi tạo PW thì yêu cầu phải có chữ hoa và chữ số. Vậy vì sao khi reset lại mật khẩu chỉ yêu cầu là ít nhất 8 kí tự
  • phần message: 'Password must be at least 5 characters long, contain at least one uppercase letter and one number'. Đồng bộ toàn bộ tiếng việt. Nửa anh nửa việt gì

Page/404/PageNotFound.tsx

  • bg-slate-950? Nhắc rồi? BẢO ĐỌC DOCS SAO KHÔNG ĐỌC. sao lại fix cứng màu? Còn màu gradient để đó
  • text-slate-400? rounded-[28px]? max-w-[30rem]? bg-red-950? shadow-red-950/30 ? hover:bg-red-900'? Còn nhiều nữa CHECK LẠI TOÀN BỘ COLOR VARIABLE

pages/login/Login.tsx

  • Đặt lại theo color variable
  • KHÔNG SET CỨNG WIDTH HEIGHT

pages/register/Register.tsx

  • Tách những hàm getPasswordStrength, getPasswordStrengthLabel, getPasswordStrengthColor ra một file riêng rồi import nó vào.
  • Lúc thì set cứng height, lúc thì set h-16??

pages/reset-password/ResetPassword.tsx

  • Trong hàm getPasswordStrength, check độ dài >= 6. Nhưng trong ResetPasswordSchema yêu cầu ít nhất 8 ký tự.
  • Hàm getPasswordStrength đặt sai chỗ: Hàm này được định nghĩa bên ngoài component nhưng lại thực hiện logic kiểm tra mật khẩu.
  • Các đoạn text như "YẾU", "TRUNG BÌNH", "MẠNH" hoặc các thông báo lỗi đang được viết trực tiếp. dự án cần đa ngôn ngữ (i18n) thì sửa lại
  • Phần hiển thị "Password Strength" nên được tách thành một Component riêng (PasswordStrengthBar.tsx) thay vì viết lồng vào giữa FormField.
  • Đúng ra nên dùng: padding: "var(--grid-margin)" từ config container để đảm bảo khoảng cách lề luôn khớp với các trang khác trong hệ thống trên mọi thiết bị.

Những file sau để ý tương tự những lỗi như thế

…the three ( getPasswordStrength, getPasswordStrengthLabel, getPasswordStrengthColor) functions into one file.
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

Successfully merging this pull request may close these issues.

2 participants