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: translate by proceeding i18n #14878

Conversation

VenkateshSircilla123
Copy link

@VenkateshSircilla123 VenkateshSircilla123 commented May 5, 2024

What does this PR do?

it translates "By proceeding, you agree to our terms and privacy policy" using i18n

/claim #14877
Fixes #14877

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Picsart_24-05-05_12-43-59-089

Mandatory Tasks

  • [ X ] Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

Copy link

vercel bot commented May 5, 2024

@VenkateshSircilla123 is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

@graphite-app graphite-app bot requested a review from a team May 5, 2024 04:49
@graphite-app graphite-app bot added the community Created by Linear-GitHub Sync label May 5, 2024
@github-actions github-actions bot added i18n area: i18n, translations 💎 Bounty A bounty on Algora.io labels May 5, 2024
Copy link
Contributor

github-actions bot commented May 5, 2024

Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link.

@dosubot dosubot bot added ✨ feature New feature or request 🐛 bug Something isn't working labels May 5, 2024
Copy link

graphite-app bot commented May 5, 2024

Graphite Automations

"Add community label" took an action on this PR • (05/05/24)

1 label was added to this PR based on Keith Williams's automation.

"Add consumer team as reviewer" took an action on this PR • (05/05/24)

1 reviewer was added to this PR based on Keith Williams's automation.

Copy link
Contributor

github-actions bot commented May 5, 2024

📦 Next.js Bundle Analysis for @calcom/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@VenkateshSircilla123 VenkateshSircilla123 changed the title Bug/translate by proceeding i18n Bug: translate by proceeding i18n May 5, 2024
@VenkateshSircilla123 VenkateshSircilla123 changed the title Bug: translate by proceeding i18n fix: translate by proceeding i18n May 5, 2024
@anikdhabal
Copy link
Contributor

No this is wrong way to fix that. I've already handled that in one of my pr.

@PeerRich
Copy link
Member

PeerRich commented May 6, 2024

we dont translate strings by breaking them up. this cannot be done for other languages

@PeerRich PeerRich closed this May 6, 2024
@VenkateshSircilla123 VenkateshSircilla123 deleted the bug/translate-by-proceeding-i18n branch May 6, 2024 07:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🙋 Bounty claim 💎 Bounty A bounty on Algora.io 🐛 bug Something isn't working community Created by Linear-GitHub Sync ✨ feature New feature or request i18n area: i18n, translations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

i18n: translate "by proceeding.."
3 participants