Limited username to 20 characters

openfoodfacts · Apr 21, 2022 · d176245 · d176245
1 parent f671807
commit d176245
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/packages/smooth_app/lib/helpers/user_management_helper.dart b/packages/smooth_app/lib/helpers/user_management_helper.dart
@@ -20,6 +20,9 @@ class UserManagementHelper {
   static bool isUsernameValid(final String username) =>
       username.isNotEmpty && _userRegex.hasMatch(username);
 
+  static bool isUsernameLengthValid(final String username) =>
+      username.length <= 20;
+
   static bool isPasswordValid(final String password) => password.length >= 6;
 }
 

diff --git a/packages/smooth_app/lib/pages/user_management/sign_up_page.dart b/packages/smooth_app/lib/pages/user_management/sign_up_page.dart
@@ -117,6 +117,10 @@ class _SignUpPageState extends State<SignUpPage> {
                     _userController.trimmedText)) {
                   return appLocalizations.sign_up_page_username_description;
                 }
+                if (!UserManagementHelper.isUsernameLengthValid(
+                    _userController.trimmedText)) {
+                  return 'Username must be less than 20 characters'; // TODO(vik4114): localization
+                }
                 return null;
               },
             ),