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

Functions #29

Merged
merged 10 commits into from Nov 15, 2019
Merged

Functions #29

merged 10 commits into from Nov 15, 2019

Conversation

ghost
Copy link

@ghost ghost commented Oct 28, 2019

I have translated all the files (assigned to me) completely. Please, check those change.

@CLAassistant
Copy link

CLAassistant commented Oct 28, 2019

CLA assistant check
All committers have signed the CLA.

@raikusy raikusy changed the title Translating Function DONE Functions Oct 29, 2019
@raikusy
Copy link
Contributor

raikusy commented Oct 29, 2019

@lifeoflikhon ধন্যবাদ আপনার কন্ট্রিবিউশন এর জন্য। পুল রিকুয়েস্ট দেয়ার সময়, পুল রিকুয়েস্ট এর টাইটেল আপনি যে চ্যাপ্টার অনুবাদ করেছেন, হুবহু সেটির টাইটেল হতে হবে, এর সাথে অন্য কিছু লেখা যাবে না। আমি এটি আপডেট করে দিয়েছি। ভবিষ্যতে খেয়াল রাখবেন। 😄

Copy link
Contributor

@raikusy raikusy left a comment

Choose a reason for hiding this comment

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

বেশ কিছু জায়গায় কোড এর ফরম্যাটিং পরিবর্তন হয়েছে, কোথাও স্পেস কেটে গেছে, কোথাও ডাবল কোট সিঙ্গেল কোট হয়ে গেছে। মুল লেখা থেকে কোডের অংশটুকু তে কোন পরিবর্তন করা যাবে না। শুধু মাত্র কমেন্ট গুলি অনুবাদ করা যেতে পারে। কিন্তু কোন প্রকার লিন্টিং/প্রিটিয়ার করা যাবে না। অনুবাদ করার সময় এডিটর এর ডিফল্ট লিন্টিং/প্রিটিয়ার ডিসেবল করে রাখতে হবে।
ধন্যবাদ।


```js
function checkAge(age) {
return (age > 18) || confirm('Did parents allow you?');
return age > 18 || confirm('Did parents allow you?');
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
return age > 18 || confirm('Did parents allow you?');
return (age > 18) || confirm('Did parents allow you?');

কোডের স্টাইল চেঞ্জ করা যাবে না।


```js
function showMessage() {
alert( 'Hello everyone!' );
alert('Hello everyone!');
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
alert('Hello everyone!');
alert( 'Hello everyone!' );

কোড স্টাইল পরিবর্তন করা যাবে না।

*!*
let message = "Hello, I'm JavaScript!"; // লোকাল ভ্যারিয়েবল
*/!*
alert(message);
Copy link
Contributor

Choose a reason for hiding this comment

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

মুল লেখার সাথে লাইন নাম্বার পরিবর্তন হয়ে গেছে। ৬৩ নাম্বার লাইন খালি থাকার কথা। এছাড়া মুল কোড এর স্টাইল ও পরিবর্তন হয়েছে। মুল কোড কে লিন্ট/প্রিটিয়ার করা যাবে না।

@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@raikusy
Copy link
Contributor

raikusy commented Nov 3, 2019

@lifeoflikhon পরিবর্তন গুলি আপডেট করে /done লিখে কমেন্ট করে জানাবেন। আমরা তাহলে পুনরায় রিভিউ করতে পারবো।
ধন্যবাদ।

@ghost
Copy link
Author

ghost commented Nov 7, 2019

/done

@raikusy raikusy merged commit 021a741 into javascript-tutorial:master Nov 15, 2019
@javascript-translate-bot

Thank you 💖 I updated the Progress Issue #1 🎉 🎉 🎉

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

Successfully merging this pull request may close these issues.

4 participants