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

Country Capital Quiz #2544 #2609

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

Conversation

Vaishnavi-Raykar
Copy link

@Vaishnavi-Raykar Vaishnavi-Raykar commented Oct 20, 2024

Closes #2544

  • Translate all labels in the App with Tolgee
  • Store the project into /demos folder
  • Add a screenshot or video demo to the PR
  • Write a short article describing your demo app on DEV.to or your blog
  • Please show your support and give a ⭐ to the Tolgee repository

Screenshots -

a

image

Links -

Deployment Link

Article Link

@Vaishnavi-Raykar Vaishnavi-Raykar changed the title Country Capital Quiz Country Capital Quiz #2544 Oct 20, 2024
@Vaishnavi-Raykar
Copy link
Author

@marketachalupnikova
@JanCizmar

Please review my PR.

@JanCizmar
Copy link
Contributor

In the dev. to article, you should be talking more about, how you build this demo and you should link to Tolgee github or website. Can you plaese fix it? https://dev.to/vaishnavi_raykar/country-capital-quiz-2cok

@JanCizmar JanCizmar self-requested a review October 21, 2024 09:37
Copy link
Contributor

@JanCizmar JanCizmar left a comment

Choose a reason for hiding this comment

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

  • Please rename the folder to Country Capital Quiz
  • The language selector is not visible on larger screens

@JanCizmar
Copy link
Contributor

Looks great! 🚀 Please, just fix the CR issues.

@Vaishnavi-Raykar
Copy link
Author

@JanCizmar Yes Sir

@Vaishnavi-Raykar
Copy link
Author

@JanCizmar , Please review sir I fixed assign issues

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

Successfully merging this pull request may close these issues.

🤩 Create World Capitals Quiz (Hacktoberfest Demo)
2 participants