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

fix(phrases): improve wording of "existing account?" question #7172

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

Conversation

nickdaugherty
Copy link

Summary

A small improvement to the wording of the "existing account" question on login prompts. Changes "Already had an account?" to "Already have an account?".

This is my first contribution to Logto so let me know if I did anything wrong!

Testing

Text-only change so didn't set up a local env

Checklist

  • .changeset
  • unit tests
  • integration tests
  • necessary TSDoc comments

Copy link

COMPARE TO master

Total Size Diff 📈 +2 Bytes

Diff by File
Name Diff
packages/phrases-experience/src/locales/en/description.ts 📈 +1 Bytes
packages/phrases-experience/src/locales/en/secondary.ts 📈 +1 Bytes

@nickdaugherty nickdaugherty force-pushed the fix/existing-account-english-wording branch 4 times, most recently from 602a354 to 7d49097 Compare March 24, 2025 01:14
@darcyYe darcyYe changed the title fix(core): Improve wording of "existing account?" question fix(core): improve wording of "existing account?" question Mar 26, 2025
@darcyYe
Copy link
Contributor

darcyYe commented Mar 26, 2025

Thanks for your contribution!

@darcyYe darcyYe requested a review from a team March 26, 2025 03:45
@darcyYe darcyYe changed the title fix(core): improve wording of "existing account?" question fix(phrases): improve wording of "existing account?" question Mar 26, 2025
@charIeszhao
Copy link
Member

The commit message should be "phrases", though

@darcyYe
Copy link
Contributor

darcyYe commented Mar 26, 2025

Our CI is reporting an error. Could you please update this commit message? For example, change it to: 'fix(phrases): improve en i18n content'.

"have" reads better in English
@nickdaugherty nickdaugherty force-pushed the fix/existing-account-english-wording branch from 7d49097 to f315c5d Compare March 26, 2025 13:26
@nickdaugherty
Copy link
Author

No problem, commit message updated. Hopefully that does the trick 🤞

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

Successfully merging this pull request may close these issues.

3 participants