-
Notifications
You must be signed in to change notification settings - Fork 89
Interaction: alert, prompt, confirm #21
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
Conversation
Please make the requested changes. After it, add a comment "/done". |
/done |
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 requested changes. After it, add a comment "/done". |
/done |
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.
I think we dont need to translate the texts inside code blocks. Isn't that mentioned in the rules ?
Please make the requested changes. After it, add a comment "/done". |
/done |
Brother @ta-riq we can change the string inside any code blocks and also we translate comments. otherwise, code blocks are not changeable. thanks for your opinions. |
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.
almost ok, thanks for your great work brother @shuvo575
Thank you 💖 I updated the Progress Issue #1 🎉 🎉 🎉 |
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.
Nice
Here you go.
:)