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

feat: add homepage translations for asyncapi website #2040

Merged
merged 5 commits into from
Aug 14, 2023

Conversation

anshgoyalevil
Copy link
Member

@anshgoyalevil anshgoyalevil commented Aug 8, 2023

Description

  • This PR adds the homepage translations in German language.
  • Cypress Tests are modified to work with the changes
  • .gitignore file is modified to ignore the generated screenshots and videos while using the command npx cypress run --component

Related issue(s)

fixes partly #2039

How to test

Live changes can be viewed by appending en or de to the netlify URL for this PR. For example {netlifyURL}/de

Signed-off-by: Ansh Goyal <[email protected]>
@netlify
Copy link

netlify bot commented Aug 8, 2023

Deploy Preview for asyncapi-website ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit d161398
🔍 Latest deploy log https://app.netlify.com/sites/asyncapi-website/deploys/64d865be0fb77100083e490c
😎 Deploy Preview https://deploy-preview-2040--asyncapi-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@anshgoyalevil
Copy link
Member Author

//cc @magicmatatjahu 🚀

Signed-off-by: Ansh Goyal <[email protected]>
@github-actions
Copy link

github-actions bot commented Aug 8, 2023

⚡️ Lighthouse report for the changes in this PR:

Category Score
🔴 Performance 22
🟢 Accessibility 98
🟢 Best practices 100
🟢 SEO 100
🔴 PWA 30

Lighthouse ran on https://deploy-preview-2040--asyncapi-website.netlify.app/

@magicmatatjahu
Copy link
Member

@anshgoyalevil Try to also translate the buttons like here

image

and also another section like Latest news and blogs

@anshgoyalevil
Copy link
Member Author

Sure. Missed out on that 🚀

@anshgoyalevil
Copy link
Member Author

anshgoyalevil commented Aug 13, 2023

@magicmatatjahu Done with the changes 🚀 Please check

@magicmatatjahu
Copy link
Member

magicmatatjahu commented Aug 14, 2023

@anshgoyalevil

And last one 😄

image

@anshgoyalevil
Copy link
Member Author

anshgoyalevil commented Aug 14, 2023

@magicmatatjahu I tried translating this section but we are importing the data for it from json file, and mapping it thereafter to the next components, so it is not getting translated. Should I transfer the content of that json to an array inside the file where it is being consumed? 🤔

@magicmatatjahu
Copy link
Member

@anshgoyalevil Yes, move that to the i18n files :)

@magicmatatjahu
Copy link
Member

I can accept that PR and you can do that in next PRs, ok? :) Merging.

@anshgoyalevil
Copy link
Member Author

Okay. Thanks 😄

@magicmatatjahu
Copy link
Member

/rtm

@asyncapi-bot asyncapi-bot merged commit ad69579 into asyncapi:master Aug 14, 2023
@anshgoyalevil anshgoyalevil deleted the homepage-t branch September 8, 2023 16:17
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