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

add new arabic courses #6964

Merged
merged 5 commits into from
Jul 27, 2022

Conversation

AhmedElTabarani
Copy link
Contributor

What does this PR do?

Add resource(s) | Remove resource(s) | Add info | Improve repo

For resources

Description

Why is this valuable (or not)?

How do we know it's really free?

For book lists, is it a book? For course lists, is it a course? etc.

Checklist:

  • Read our contributing guidelines
  • Search for duplicates.
  • Include author(s) and platform where appropriate.
  • Put lists in alphabetical order, correct spacing.
  • Add needed indications (PDF, access notes, under construction)

Follow-up

  • Check the status of GitHub Actions and resolve any reported warnings!

Copy link
Member

@LuigiImVector LuigiImVector left a comment

Choose a reason for hiding this comment

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

Missed some ‏

Also remember that atomic commits are prefered:

- prefer atomic commits (one commit by addition/deletion/modification) over bigger commits. No need to squash your commits before submitting a PR. (We will never enforce this rule as it's just a matter of convenience for the maintainers)

* [Data Structure C++‎‏](https://www.youtube.com/playlist?list=PLsGJzJ8SQXTcsXRVviurGei0lf_t_I4D8) - Mega Code
* [Data Structures Full Course In Arabic‏](https://www.youtube.com/playlist?list=PLCInYL3l2AajqOUW_2SwjWeMwf4vL4RSp) - Adel Nasim
* [Grokking Algorithms](https://www.youtube.com/playlist?list=PLWCr3KJclfzGV82gewnuAoKFBt_f9FvNa) - Programming Books
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* [Grokking Algorithms](https://www.youtube.com/playlist?list=PLWCr3KJclfzGV82gewnuAoKFBt_f9FvNa) - Programming Books
* [Grokking Algorithms‏](https://www.youtube.com/playlist?list=PLWCr3KJclfzGV82gewnuAoKFBt_f9FvNa) - Programming Books

@@ -113,6 +117,7 @@
* [CS Master - Level 3 - Databases قواعد البيانات](https://www.youtube.com/playlist?list=PLL2zWZTDFZzhXQ1bcYlO3PtN4MsLiG-gy) - KMR Script
* [Database 1 - المقرر النظرى - Fundamentals of Database Systems](https://www.youtube.com/playlist?list=PL37D52B7714788190) - محمد الدسوقى
* [Database Design](https://www.youtube.com/playlist?list=PLkzDzmo9y3VHDFKp7LuXd-FwbefvTL5o0) - تخاريف مبرمج
* [Designing Data Intensive Applications](https://www.youtube.com/playlist?list=PLTRDUPO2OmIljJwE9XMYE_XEgEIWZDCuQ) - Ahmed Elemam
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* [Designing Data Intensive Applications](https://www.youtube.com/playlist?list=PLTRDUPO2OmIljJwE9XMYE_XEgEIWZDCuQ) - Ahmed Elemam
* [Designing Data Intensive Applications‏](https://www.youtube.com/playlist?list=PLTRDUPO2OmIljJwE9XMYE_XEgEIWZDCuQ) - Ahmed Elemam

@@ -170,6 +175,7 @@
* [Learn SASS In Arabic 2021‏](https://www.youtube.com/playlist?list=PLDoPjvoNmBAzlpyFHOaB3b-eubmF0TAV2) - Elzero Web School
* [Sass‏](https://www.youtube.com/playlist?list=PLDoPjvoNmBAz6bF7qObm2a1mLN_WHAWQo) - Elzero Web School
* [Sass - Create a website‏](https://www.youtube.com/playlist?list=PLDoPjvoNmBAz9sluuyOWPifXvySgrGma8) - Elzero Web School
* [Yonisfy CS94's | Web Programming with JavaScript](https://www.youtube.com/playlist?list=PLLWuK602vNiXqkUqcoKrVjCyOrZh7Tw0e) - Mohammed Elzanaty
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* [Yonisfy CS94's | Web Programming with JavaScript](https://www.youtube.com/playlist?list=PLLWuK602vNiXqkUqcoKrVjCyOrZh7Tw0e) - Mohammed Elzanaty
* [Yonisfy CS94's | Web Programming with JavaScript‏](https://www.youtube.com/playlist?list=PLLWuK602vNiXqkUqcoKrVjCyOrZh7Tw0e) - Mohammed Elzanaty

* [كورس Ajax بالعربي-ITI‏](https://www.youtube.com/playlist?list=PLzCpl3aBwaY7wv9ZZkhH7lZELpz_fP81N) - Mobarmg
* [كورس ES6 بالعربي-ITI‏](https://www.youtube.com/playlist?list=PLzCpl3aBwaY5nQxWH0HaNibR9UXKICWxz) - Mobarmg
* [Arabic JavaScript‏](https://www.youtube.com/playlist?list=PLL2zWZTDFZzgU2x6Kk6w0qx5piLgMODsm) - KMR Script
* [Arabic JavaScript ES6 الاصدار السادس من الجافاسكربت](https://www.youtube.com/playlist?list=PLL2zWZTDFZzilx_LJ_mCRDETtDOyBg0UT) - KMR Script
* [Complete Intro to Javascript‏](https://www.youtube.com/playlist?list=PLLWuK602vNiU-kIpNuw5Z7HRbV4pBaHlL) - Mohammed Elzanaty
* [Design Patterns \| javascript \| [Arabic]‏](https://www.youtube.com/playlist?list=PLS-MrzRLZtmduTfp_bReagQKg7I-GVr-5) - Shadow Coding
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* [Design Patterns \| javascript \| [Arabic]‏](https://www.youtube.com/playlist?list=PLS-MrzRLZtmduTfp_bReagQKg7I-GVr-5) - Shadow Coding
* [Design Patterns \| javascript \| \[Arabic\]‏](https://www.youtube.com/playlist?list=PLS-MrzRLZtmduTfp_bReagQKg7I-GVr-5) - Shadow Coding

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The square brackets must be escaped also ?

Copy link
Member

@LuigiImVector LuigiImVector Jul 25, 2022

Choose a reason for hiding this comment

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

could create conflicts with those in the link

@@ -336,6 +350,8 @@
* [Clean Code Book - بالعربي](https://www.youtube.com/playlist?list=PLwWuxCLlF_ufTMlXoJlQvImqz9wIfcWsX) - Omar Ahmed
* [Creational Design Patterns بالعربى](https://www.youtube.com/playlist?list=PLnqAlQ9hFYdewk9UKGBcHLulZNUBpNSKJ) - Mohammed Reda
* [Declarative Programming](https://www.youtube.com/playlist?list=PLpbZuj8hP-I6F-Zj1Ay8nQ1rMnmFnlK2f) - درة الاكواد لابن حماد
* [Head First Object-Oriented Analysis and Design‏](https://www.youtube.com/playlist?list=PLWCr3KJclfzGkG7CXBmSdlIcIpjxo4FbF) - Programming Books
* [Software Design Patterns](https://www.youtube.com/playlist?app=desktop&list=PLrwRNJX9gLs3oQyBoXtYimY7M5aSF0_oC) - محمد يحيى
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* [Software Design Patterns](https://www.youtube.com/playlist?app=desktop&list=PLrwRNJX9gLs3oQyBoXtYimY7M5aSF0_oC) - محمد يحيى
* [Software Design Patterns‏](https://www.youtube.com/playlist?app=desktop&list=PLrwRNJX9gLs3oQyBoXtYimY7M5aSF0_oC) - محمد يحيى

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No need to add ‏ here
the author name in Arabic, so the context will not swap

Copy link
Contributor Author

Choose a reason for hiding this comment

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

image

Copy link
Member

Choose a reason for hiding this comment

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

I didn't know that, could you add it to #6964?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

#6964 ? You miss the issue?

Copy link
Contributor Author

@AhmedElTabarani AhmedElTabarani Jul 25, 2022

Choose a reason for hiding this comment

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

If you mean this #6714, this only for problems

* [name in english](url) - name in arabic
That not cause any problem

Copy link
Member

Choose a reason for hiding this comment

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

If you mean this #6714

ops, i confused id

That not cause any problem

maybe you could add it just for clarification, in an "extra case 2"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

maybe you could add it just for clarification, in an "extra case 2"

done

@LuigiImVector LuigiImVector added waiting for changes PR has been reviewed and changes/suggestions requested 🗣️ locale:ar Resources addressing "Arabic / العربية" language labels Jul 25, 2022
@AhmedElTabarani AhmedElTabarani mentioned this pull request Jul 25, 2022
7 tasks
@AhmedElTabarani
Copy link
Contributor Author

I will delete Head First Object-Oriented Analysis and Design, Grokking Algorithms, Designing Data Intensive Applications

Because it does courses to explain paid books as @eshellman said here #6764

@AhmedElTabarani
Copy link
Contributor Author

@eshellman, relying on a paid book to explain is illegal? Or explain the book itself is illegal?

Because most of the courses depend on paid books such as clean code courses

@eshellman
Copy link
Collaborator

What do you think - Is it a free course if the book isn't free? Many courses are taught so they don't really need the book, but maybe these really rely heavily on the not-free book? The test is: does the course stand on its own? We have to rely on you to make the judgement.

@eshellman eshellman requested a review from LuigiImVector July 26, 2022 06:48
@LuigiImVector LuigiImVector added 🚀 ready to merge LGTM. Waiting for final approval and removed waiting for changes PR has been reviewed and changes/suggestions requested labels Jul 26, 2022
@eshellman eshellman merged commit 19958fb into EbookFoundation:main Jul 27, 2022
@AhmedElTabarani AhmedElTabarani deleted the add-arabic-courses branch July 27, 2022 18:34
@@ -170,6 +173,7 @@
* [Learn SASS In Arabic 2021‏](https://www.youtube.com/playlist?list=PLDoPjvoNmBAzlpyFHOaB3b-eubmF0TAV2) - Elzero Web School
* [Sass‏](https://www.youtube.com/playlist?list=PLDoPjvoNmBAz6bF7qObm2a1mLN_WHAWQo) - Elzero Web School
* [Sass - Create a website‏](https://www.youtube.com/playlist?list=PLDoPjvoNmBAz9sluuyOWPifXvySgrGma8) - Elzero Web School
* [Yonisfy CS94's | Web Programming with JavaScript‏](https://www.youtube.com/playlist?list=PLLWuK602vNiXqkUqcoKrVjCyOrZh7Tw0e) - Mohammed Elzanaty
Copy link
Contributor Author

@AhmedElTabarani AhmedElTabarani Jul 30, 2022

Choose a reason for hiding this comment

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

I forget escape |, we need a regex to check these mistakes if a '|' have an escape character '\' or not, or something like this.

Copy link
Member

Choose a reason for hiding this comment

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

This issue is discussed on #5176

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 ready to merge LGTM. Waiting for final approval 🗣️ locale:ar Resources addressing "Arabic / العربية" language
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants