linuxscout
Feature/reafactor add pwa
لققد قمت بالتالئ اولا: تحويل واجهة المستخدم الى react ثانيا: اضافة service worker و بعض ال caching للصور و الملفات ثالثا: تعدبل على لعض اسماء الملفات
كان من الأفضل ألا تقوم بالتغيير الجذري للمشروع أقترح عليك الاتي :
- قم بالإضافة ملف جديد في Interface > react و ضع ملفاتك الخاصة فيه
- إحترم API المستعملة حاليا ( تغيير API الأن قد يسبب تعطل بعض التطبيقات التي تستعمل هذه الخدمة )
- حاول ألا تغير في الملفات قدر الإمكان, لا تقم بالحذف أي ملف أي خاصية جديدة غير متعلقة بالتحديثات الأخرى (independent) من الافضل أن تضعها في PR منفصلة
We appreciate your contribution for the project.
@tarek-berkane
سكرا جزيلا
اذا سابدؤ العمل من جديد انا اقترحت ان نستخدم نفس واجهة الوب على الدسكتوب و ذلك باضافة خاصية PWA
كما اني اريد اضافة بعض التحسينات على api وذلك بانشاء v2 فقد رايت ان بعض الروابط لاتحترم بعض ال standards في ال api design فبعضها تستخدم افعالا في الروابط في حين انه يجب علينا استخدام اسماء فقط في الروابط عند انشاء rest api
ملاحظة اخرى و هي انه يمكننا اضافة linter و coding rules و عمل refactoring لبعض الاجزاء من الكود لجعله انضف و اسهل فهما
ارجو ان تقدمو لي رايكم فيما ذكرت و شكرا