General Settings - Character limit counter and hint to TextInput are not displayed for name field

[kbecciv]

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!

---

Action Performed:

1. Launch App as login user
2. Navigate to Workspace > General settings
3. Clear field and start typing name

Expected Result:

Character limit counter and hint to TextInput are displayed for name field

Actual Result:

Character limit counter and hint to TextInput are not displayed for name field

Workaround:

Unknown

Platform:

Where is this issue occurring?

• Web
• iOS
• Android
• Desktop App
• Mobile Web

Version Number: 1.1.39-1

Reproducible in staging?: Yes

Reproducible in production?: Yes

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation

Bug5455126_Counter.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause

Slack conversation:

Issue was found when executing : #7577

View all open jobs on GitHub

[MelvinBot]

Triggered auto assignment to @iwiznia (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[iwiznia]

Do we limit the characters the policy name can have? If not, then I think this is not an issue and we should close this.

[iwiznia]

I don't see any limit on length of name, so this is not an issue.

[isagoico]

@iwiznia This issue is failing QA for PR #7577.
Reopening the issue.
CC @parasharrajat

[iwiznia]

This input does not have any limit, so I don't see why we would show the counter. Can you explain?

[parasharrajat]

Yeah, I agree with @iwiznia. @iwiznia I added the new feature on #7577 but it is not added to any screens yet.

[Julesssss]

Hi @isagoico, I'm going to close this again and mark it off the list as this is expected behavior. I don't see how it is failing #7577 honestly.

Is there a test case that requires that we show the limit error? If so I think we can remove that from the test suit for now.

[isagoico]

Oh I got it now, my apologies I misread the QA steps and was confused with the testing screenshots. We usually compare the testing screenshots to verify the intended behavior and the ones added in the PR show the counter. Closing the issue then.