Skip to content

fix: validate user about bio to max 255 characters - #348

Merged
trtajim merged 1 commit into
mainfrom
fix/user-about-max-length
Sep 19, 2026
Merged

trtajim merged 1 commit into
mainfrom
fix/user-about-max-length

Conversation

@trtajim

@trtajim trtajim commented Sep 19, 2026

Copy link
Copy Markdown
Member

Description

Fixes database truncated string exception (SQLSTATE[22001]: String data, right truncated: 1406 Data too long for column 'about') when user about/bio exceeds 255 characters.

Changes

  • Updated about validation rule from max:1000 to max:255 across:
    • UpdateProfileRequest
    • StoreUserRequest
    • UpdateUserRequest
  • Added maxlength="255" attribute and real-time character counter (0/255) in:
    • Profile.vue
    • admin/users/CreateOrEdit.vue
  • Added feature test in UserProfileTest.php to verify 255-character limit enforcement.

@coderabbitai

coderabbitai Bot commented Sep 19, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Repository: hscstack/platform/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 06639f86-6324-4b81-8b9c-a2d158594a67

📥 Commits

Reviewing files that changed from the base of the PR and between 05fac3c and 6eb4a6c.

📒 Files selected for processing (6)
  • app/Http/Requests/Profile/UpdateProfileRequest.php
  • app/Http/Requests/User/StoreUserRequest.php
  • app/Http/Requests/User/UpdateUserRequest.php
  • resources/js/pages/Profile.vue
  • resources/js/pages/admin/users/CreateOrEdit.vue
  • tests/Feature/UserProfileTest.php
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@trtajim
trtajim merged commit 68c074a into main Sep 19, 2026
5 of 6 checks passed
@trtajim
trtajim deleted the fix/user-about-max-length branch September 19, 2026 17:16
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.

1 participant