Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added password placeholders in all .i18n.json files. Please check if … #34041

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

aadi-joshi
Copy link

@aadi-joshi aadi-joshi commented Nov 23, 2024

fix(i18n): add password placeholder for all i18n files

I have added a passwordplaceholder for all the .i18n.json files. This change globally adds placeholders for the password fields. While I faced some difficulty in reproducing the issue on my local setup, this modification should fix the issue as intended.

Issue(s):

This was the issue I’m trying to fix:
#34032

Steps to test or reproduce

The 'login' page should now effectively show a placeholder for the password form as 8 dots.

Further comments:

This is my first commit, and I would appreciate guidance on setting up the local environment to contribute more effectively to Rocket.Chat. I’m excited to continue contributing.

…this fixed the 'no placeholder' under password issue.
Copy link
Contributor

dionisio-bot bot commented Nov 23, 2024

Looks like this PR is not ready to merge, because of the following issues:

  • This PR is missing the 'stat: QA assured' label
  • This PR is missing the required milestone or project
  • This PR has an invalid title

Please fix the issues and try again

If you have any trouble, please check the PR guidelines

@CLAassistant
Copy link

CLAassistant commented Nov 23, 2024

CLA assistant check
All committers have signed the CLA.

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

Successfully merging this pull request may close these issues.

3 participants