-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Translation of Quiz into Hindi #85
Translation of Quiz into Hindi #85
Conversation
] | ||
}, | ||
{ | ||
"questionText": "सर्वरलेस कोड केवल जावास्क्रिप्ट और कोबोल में लिखा जा सकता है:", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There seems to be some error here can you check if this?
}, | ||
{ | ||
"questionText": "स्व-हस्ताक्षरित X.509 प्रमाण पत्र उत्पादन वातावरण के लिए आदर्श हैं", | ||
"answerOptions": [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please change it in English language as thats mostly used in day to day life
"quiz": [ | ||
{ | ||
"questionText": "सममित कुंजी एन्क्रिप्शन असममित की एन्क्रिप्शन की तुलना में होती है जिसमें तरीके:", | ||
"answerOptions": [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please edit this question options and heading and chnge it to more understandable form by converting it in english or easier hindi
Hi, I didn't go through the whole file but one thing I spotted was that you have translated true false to सही गलत. You are doing a great job otherwise, @innovation-platform. Kudos |
Thanks for mentioning. I'll correct it. |
Please ignore the second commit. |
Sorry for these may commits. Please review the recent commit i.e "made the changes to the hin.json file as suggested". Thank You. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please make the required changes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@AdityaGarg00, @innovation-platform, @EliteWarrior315
Some changes are required. While everything else is fine :)
@anurag-0-1-a - are the changes you mention done? Can this be merged? |
No @jimbobbennett. @innovation-platform Kindly make the mentioned changes, please. |
I am so sorry for opening another pull request as I faced some issues while pushing.
I have made the suggested changes.