Skip to content

Apoorva - feat: Add 100 character limit to team name inputs#4565

Merged
one-community merged 1 commit intodevelopmentfrom
feature/limit-team-name-100-chars
Jan 11, 2026
Merged

Apoorva - feat: Add 100 character limit to team name inputs#4565
one-community merged 1 commit intodevelopmentfrom
feature/limit-team-name-100-chars

Conversation

@apoorvajainrp21
Copy link
Copy Markdown
Contributor

@apoorvajainrp21 apoorvajainrp21 commented Dec 16, 2025

  • Added maxLength attribute to team name input in CreateNewTeamPopup
  • Added character counter with visual feedback in both locations
  • Added maxLength to AddTeamsAutoComplete component
  • Enforced 100 character limit in both team creation flows
  • Tested in both locations: Other Links > Teams and Profile > Teams

Description

This PR implements a 100-character limit for team name inputs across both team creation locations in the application. The implementation includes visual feedback through character counters and enforces the limit both at the browser level (using HTML maxLength attribute) and programmatically through validation logic.

Related PRS (if any):

This frontend PR is related to PR #1229
To test this frontend PR, you need to checkout the Ramakrishna_bugfix_assign_members_new_team backend branch.

Main changes explained:

Updated src/components/Teams/CreateNewTeamPopup.jsx:

Added TEAM_NAME_MAX_LENGTH = 100 constant
Added maxLength attribute to the Input component
Implemented real-time character counter displaying "X/100 characters"
Updated handleTeamNameChange function to enforce character limit programmatically
Added dark mode support for character counter styling

Updated src/components/UserProfile/TeamsAndProjects/AddTeamsAutoComplete.jsx:

Added TEAM_NAME_MAX_LENGTH = 100 constant
Added maxLength attribute to the Input component
Implemented real-time character counter with "X/100 characters" display
Added dark mode support for character counter styling

How to test:

Setup:

Checkout current branch: feature/limit-team-name-100-chars
Run npm install to install dependencies
Checkout backend branch Ramakrishna_bugfix_assign_members_new_team in HGNRest repository
Start both backend and frontend servers
Clear site data/cache in browser

Test Location 1 - Other Links → Teams:

Log in as Admin/Owner user
Navigate to Dashboard → Other Links → Teams → Create New Team
Click "Create New Team" button to open the modal
In the team name input field:

Verify character counter shows "0/100 characters"
Type a normal team name (e.g., "Test Team 2024") - verify counter updates
Try to type more than 100 characters - verify it stops at 100
Try pasting text longer than 100 characters - verify it truncates at 100
Verify counter displays correctly in both light and dark modes

Attempt to create a team with exactly 100 characters - verify it succeeds
Verify the character counter styling is appropriate and readable

Test Location 2 - Profile → Teams:

Navigate to Profile → Teams
Click "Assign Team" button to open the "Add Team" modal
In the "Add to Team" search/input field:

Verify character counter shows "0/100 characters"
Type a new team name that doesn't exist (e.g., "Brand New Team XYZ")
Verify counter updates as you type
Try to type more than 100 characters - verify it stops at 100
Try pasting text longer than 100 characters - verify it truncates at 100
Verify "Create new team: [team name]" option appears
Click to create the new team with 100 characters - verify it succeeds
Verify counter displays correctly in both light and dark modes

Screenshots or videos of changes:

Screenshot 2025-12-15 165553 Screenshot 2025-12-15 165631

Note:

Include the information the reviewers need to know.

- Added maxLength attribute to team name input in CreateNewTeamPopup
- Added character counter with visual feedback in both locations
- Added maxLength to AddTeamsAutoComplete component
- Enforced 100 character limit in both team creation flows
- Tested in both locations: Other Links > Teams and Profile > Teams
- Addresses PR #1229 requirements
@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 16, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit d715570
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6941091e5e935a0008760ad2
😎 Deploy Preview https://deploy-preview-4565--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud
Copy link
Copy Markdown

@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Jan 2, 2026
@one-community one-community changed the title feat: Add 100 character limit to team name inputs Apoorva - feat: Add 100 character limit to team name inputs Jan 2, 2026
Copy link
Copy Markdown
Contributor

@ShreyaMP1999 ShreyaMP1999 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @apoorvajainrp21
I verified locally as Admin/Owner. Tested character counter behavior in both Other Links → Teams (Create New Team) and Profile → Teams (Assign Team) flows. Confirmed real-time counter accuracy (0/100), enforcement of the 100-character limit for typing and pasting, successful creation of teams with exactly 100 characters, correct truncation beyond the limit, and proper rendering/styling in both light and dark modes. All tested scenarios are working as expected.

Screenshot 2026-01-03 at 2 58 39 PM Screenshot 2026-01-03 at 2 58 47 PM Screenshot 2026-01-03 at 3 00 00 PM Screenshot 2026-01-03 at 3 01 16 PM Screenshot 2026-01-03 at 3 04 58 PM Screenshot 2026-01-03 at 3 05 30 PM Screenshot 2026-01-03 at 3 06 37 PM

Copy link
Copy Markdown

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Apoorva,

I have reviewed your PR locally and it works as per requirement for all instances.
Screenshot 2026-01-07 at 4 59 03 PM
Screenshot 2026-01-07 at 4 59 33 PM
Screenshot 2026-01-07 at 5 00 29 PM
Screenshot 2026-01-07 at 5 00 33 PM
Screenshot 2026-01-07 at 5 03 02 PM
Screenshot 2026-01-07 at 5 03 33 PM
Screenshot 2026-01-07 at 5 04 02 PM
Screenshot 2026-01-07 at 5 04 09 PM

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit 3600bcd into development Jan 11, 2026
10 checks passed
Copy link
Copy Markdown

@Vikas-8055 Vikas-8055 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hi apoorva i have reviewed your PR and it works as per the requirements

Screenshot 2026-01-16 at 4 56 56 PM Screenshot 2026-01-16 at 4 57 44 PM Screenshot 2026-01-16 at 4 59 07 PM Screenshot 2026-01-16 at 4 59 16 PM

Copy link
Copy Markdown
Contributor

@Shravan-neelamsetty Shravan-neelamsetty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Apoorva, I’ve reviewed and tested this PR locally, and all the functionalities are working perfectly as expected. I verified the 100-character limit enforcement in both team creation flows (Other Links → Teams and Profile → Teams), including real-time character counter updates, correct handling of typing and pasting beyond the limit, successful creation with exactly 100 characters, and proper truncation thereafter. The visual feedback is clear and consistent, and the counter renders correctly in both light and dark modes.
Screenshot 2026-01-16 at 6 46 50 PM
Screenshot 2026-01-16 at 6 47 06 PM
Screenshot 2026-01-16 at 6 47 53 PM
Screenshot 2026-01-16 at 6 49 59 PM
Screenshot 2026-01-16 at 6 50 17 PM
Screenshot 2026-01-16 at 6 50 36 PM

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

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants