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

chore: add Telugu and Hindi translation of "Base is for everyone" on homepage #1286

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

Deveshb15
Copy link

@Deveshb15 Deveshb15 commented Nov 22, 2024

What changed? Why?
Adding Telugu and Hindi to our "Base is for everyone" slider.

To replicate this PR:

Clone the repo and follow steps in Getting Started
Open apps/web/src/components/base-org/root/SlidingTextSection/index.tsx
Modify text by adding your language to the end of the line, followed by " - "
Notes to reviewers

How has it been tested?

Run yarn workspace @app/web dev
Navigate to http://localhost:3000 in your browser
Make sure the slider is working as expected

Copy link

vercel bot commented Nov 22, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
base-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 20, 2024 9:25am
web-base-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 20, 2024 9:25am

Copy link

vercel bot commented Nov 22, 2024

Someone is attempting to deploy a commit to the Coinbase Team on Vercel.

A member of the Team first needs to authorize it.

@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Nov 22, 2024

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/2
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 1
Sum 2

@wbnns
Copy link
Member

wbnns commented Dec 6, 2024

@Deveshb15

Hello, thanks for contributing to Base! Could you please resolve the merge conflict?

@wbnns wbnns self-assigned this Dec 6, 2024
@wbnns wbnns added change(s) requested One or more changes have been requested base ecosystem review This is ecosystem related community contribution and removed base ecosystem review This is ecosystem related labels Dec 6, 2024
@Deveshb15
Copy link
Author

Hey @wbnns resolved the conflict, thanks

@@ -8,8 +8,7 @@ import { CSSProperties, useRef } from 'react';
export default function SlidingTextSection() {
const containerRef = useRef<HTMLDivElement>(null);

const text = ' Base is for everyone - بایس للجمیع - Base es para todos - 基地适合所有人 - La Base è per tutti - Base est pour tout le monde - Base ni ya kila mtu - Base yɛ ma obiara - Base nye amesiame tɔ - Base ji he ni mɔ fɛɛ mɔ yɔɔ - ';

const text = ' Base is for everyone - بایس للجمیع - Base es para todos - 基地适合所有人 - La Base è per tutti - Base est pour tout le monde - Base ni ya kila mtu - Base yɛ ma obiara - Base nye amesiame tɔ - Base ji he ni mɔ fɛɛ mɔ yɔɔ - బేస్ అందరి కోసం - बेस सबके लिए है -';
Copy link
Member

Choose a reason for hiding this comment

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

There should be a space at the end of the line after - and before '

@wbnns
Copy link
Member

wbnns commented Dec 19, 2024

@Deveshb15 Thank you for the updates; requested and additional change in-line 🙏

@wbnns wbnns added base web review To be reviewed by the Base web team base translation review To be reviewed by the Base translation team and removed base web review To be reviewed by the Base web team labels Dec 19, 2024
@Deveshb15
Copy link
Author

Hey @wbnns made all the changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
base translation review To be reviewed by the Base translation team change(s) requested One or more changes have been requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants