-
Notifications
You must be signed in to change notification settings - Fork 528
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Browse files
Browse the repository at this point in the history
## Explanation Fixes #4641 Fixes #4674 Fixes #4579 Fixes #4591 Miscellaneous fixes for Beta MR2. Specifically: - #4641 is addressed by updating the terms of service wording. No test is needed here since manual verification is sufficient. - #4674 is addressed by replacing the existing automatic topic updates platform with a generic "all downloads" one (which will be used for all downloads gating moving forward). Admin controls, add profile, and profile edit pages have been updated to only show downloads-related settings if this flag is enabled. Corresponding tests have been updated. FAQ question 8 was also removed since it asks about how to download lessons, and this support isn't in the app yet. Some strings referencing downloads have also been updated. - #4579 is addressed by properly handling the case where unpublish topics with previous progress would cause the topic list to fail to load. A test was added to verify the fix, but it required a new FakeAssetRepository to be introduced. - #4591 is addressed by fixing FractionInputHasIntegerPartEqualToRuleClassifierProvider's input type (which should be SIGNED_INT but was incorrectly NON_NEGATIVE_INT before; this seems to have been wrong since the classifier was introduced). Test exemption: FakeAssetRepository doesn't have a test added since it uses a real production AssetRepository, and tests don't yet exist for that). They ought to be tested together. ## Essential Checklist - [x] The PR title and explanation each start with "Fix #bugnum: " (If this PR fixes part of an issue, prefix the title with "Fix part of #bugnum: ...".) - [x] Any changes to [scripts/assets](https://github.com/oppia/oppia-android/tree/develop/scripts/assets) files have their rationale included in the PR explanation. - [x] The PR follows the [style guide](https://github.com/oppia/oppia-android/wiki/Coding-style-guide). - [x] The PR does not contain any unnecessary code changes from Android Studio ([reference](https://github.com/oppia/oppia-android/wiki/Guidance-on-submitting-a-PR#undo-unnecessary-changes)). - [x] The PR is made from a branch that's **not** called "develop" and is up-to-date with "develop". - [x] The PR is **assigned** to the appropriate reviewers ([reference](https://github.com/oppia/oppia-android/wiki/Guidance-on-submitting-a-PR#clarification-regarding-assignees-and-reviewers-section)). ## For UI-specific PRs only Screenshots showing the before & after changes due to hiding downloads options: | | Without changes | With changes | |------------------|-----------------------------------------------------------------------------------------------------------------|-----------------------------------------------------------------------------------------------------------------| | Profile creation | ![image](https://user-images.githubusercontent.com/12983742/200823396-bd12d392-a2f4-4881-b12a-b4dc4168d078.png) | ![image](https://user-images.githubusercontent.com/12983742/200821992-9ec50696-80f3-47d7-aa13-515f7317ab9a.png) | | Profile edit | ![image](https://user-images.githubusercontent.com/12983742/200823563-694a82e8-67e7-4568-b4f0-7a17967f82b1.png) | ![image](https://user-images.githubusercontent.com/12983742/200822179-8fc84d83-7add-49f4-a2db-6ddb8c7150eb.png) | | Admin controls | ![image](https://user-images.githubusercontent.com/12983742/200823499-3e5f5e92-db98-455a-a8e6-56b26b562554.png) | ![image](https://user-images.githubusercontent.com/12983742/200822125-7243faad-78e1-40e6-8c09-c0610c7a01f6.png) | Espresso test rules for affected UI tests: | Affected Test | Test Results | |-----------------------------------|--------------------------------------------------------------------------------------------------------------------------------------------------------------------------| | AdministratorControlsActivityTest | (Skipped since there are tablet tests to also run, so the suite won't actually pass cleanly) | | ProfileEditActivityTest | (Same as above) | | AdministratorControlsFragmentTest | ![oppia_assorted_beta_fixes_passing_admin_controls_fragment_test](https://user-images.githubusercontent.com/12983742/200824315-2f54ee80-570e-45f9-85ce-a2c094bf5058.png) | | ProfileEditFragmentTest | ![oppia_assorted_beta_fixes_passing_profile_edit_fragment_test](https://user-images.githubusercontent.com/12983742/200824336-949122e8-f718-4a02-9f13-5559ba8a2726.png) | New Robolectric tests failing for corresponding regressions: | Issue | Test Suite | Failure Specifics | Failure Results | |-------|--------------------------------------------------------------|-------------------|---------------------------------------------------------------------------------------------------------------------------------------| | #4579 | TopicListControllerTest | 1/36 tests fail | ![failing_topic_list_test](https://user-images.githubusercontent.com/12983742/200827182-dde7ab82-f6e2-4e58-b116-073653673441.png) | | #4591 | FractionInputHasIntegerPartEqualToRuleClassifierProviderTest | 22/23 tests fail | ![oppia_failing_fraction_test](https://user-images.githubusercontent.com/12983742/200826395-e0c384ab-920c-45b5-be88-82b19c3bfed0.png) | Missing artifacts: - Accessibility: not particularly relevant to these changes since the user-facing changes are either crash fixes or removing UI elements (except in the case of terms of service, but that doesn't require additional a11y testing coverage or demonstration here). - RTL: nothing in this change affects RTL specifically. - Landscape: since the main UI changes are removing elements, the changes to tablet aren't very "interesting" to show off. - Tablet: same rationale as for landscape. - Internationalization: same rationale as RTL.
- Loading branch information
1 parent
69c9a2b
commit 76bf81a
Showing
32 changed files
with
447 additions
and
227 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -510,7 +510,6 @@ | |
<string name="faq_question_5">من هو المشرف؟</string> | ||
<string name="faq_question_6">لماذا لا يتم تحميل مشغل الاستكشاف؟</string> | ||
<string name="faq_question_7">لماذا لا يتم تشغيل الصوت الخاص بي؟</string> | ||
<string name="faq_question_8">كيف يمكنني تنزيل موضوع؟</string> | ||
<string name="faq_question_9">لا أجد سؤالي هنا. ماذا الان؟</string> | ||
<string name="faq_answer_1"><p>إذا كانت هذه هي المرة الأولى التي تنشئ فيها ملفًا شخصيًا وليس لديك رقم تعريف شخصي: </p> <p> 1. من منتقي الملف الشخصي ، اضغط على<strong> قم بإعداد ملفات تعريف متعددة</strong></p><p> 2. قم بإنشاء رقم تعريف شخصي و<strong>احفظ</strong></p> <p> 3. املأ جميع البيانات للملف الشخصي.</p> <ol> <li>(اختياري) قم بتحميل صورة.</li> <li>إدخال اسم.</li> <li>(اختياري) قم بتعيين رقم تعريف شخصي مكون من 3 أرقام.</li> </ol> <p> 4. اضغط<strong>إنشاء</strong> . تمت إضافة هذا الملف الشخصي إلى منتقي ملف التعريف الخاص بك!<br/> <br/> إذا قمت بإنشاء ملف تعريف من قبل ولديك رقم تعريف شخصي:</p><p> 1. من منتقي الملف الشخصي ، اضغط على<strong>إضافة الملف الشخصي</strong></p> <p> 2. أدخل رقم التعريف الشخصي الخاص بك وانقر فوق<strong>إرسال</strong></p> <p>3. املأ جميع الحقول للملف الشخصي.</p> <ol> <li>(اختياري) قم بتحميل صورة.</li> <li>إدخال اسم.</li> <li>(اختياري) قم بتعيين رقم تعريف شخصي مكون من 3 أرقام.</li> </ol> <p> 4. اضغط<strong>إنشاء</strong> . تمت إضافة هذا الملف الشخصي إلى منتقي ملف التعريف الخاص بك!<br/><br/> ملاحظة: فقط ال<u>مدير</u> قادر على إدارة الملفات الشخصية.</p></string> | ||
<string name="faq_answer_2"><p>بمجرد حذف ملف التعريف:</p><p><br></p> <p><li>لا يمكن استعادة ملف التعريف.</li></p><p><li>سيتم حذف معلومات الملف الشخصي مثل الاسم والصور والتقدم بشكل دائم.</li></p><p><br></p><p>لحذف ملف تعريف (باستثناء<u>المسؤول</u></p><p>1. من الصفحة الرئيسية للمسؤول ، اضغط على زر القائمة أعلى اليسار.</p><p>2. اضغط على<strong>ضوابط المسؤول</strong></p><p>3. اضغط على<strong>تحرير ملفات التعريف</strong></p><p>4. اضغط على الملف الشخصي الذي ترغب في حذفه.</p><p>5. في الجزء السفلي من الشاشة ، انقر فوق<strong>حذف الملف الشخصي</strong></p><p>6. اضغط<strong>حذف</strong>لتأكيد الحذف.</p><p><br></p><p>ملاحظة:<u>المسؤول</u>فقط هو القادر على إدارة الملفات الشخصية.</p></string> | ||
|
@@ -519,7 +518,6 @@ | |
<string name="faq_answer_5"><p>المشرف هو المستخدم الرئيسي الذي يدير ملفات التعريف والإعدادات لكل ملف تعريف على حسابه. هم على الأرجح والدك أو معلمك أو وصي عليك الذي أنشأ هذا الملف الشخصي لك.</p><p><br></p><p>يمكن للمسؤولين إدارة الملفات الشخصية وتعيين أرقام التعريف الشخصية وتغيير الإعدادات الأخرى ضمن حساباتهم. بناءً على ملف التعريف الخاص بك ، قد تكون أذونات المسؤول مطلوبة لبعض الميزات مثل تنزيل الموضوعات وتغيير رقم التعريف الشخصي وغير ذلك.</p><p><br></p><p>لمعرفة من هو المسؤول لديك ، انتقل إلى منتقي الملف الشخصي. الملف الشخصي الأول المدرج ولديه \"المسؤول\" مكتوب باسمه هو المسؤول.</p></string> | ||
<string name="faq_answer_6"><p>إذا لم يتم تحميل مشغل الاستكشاف</p><p><br></p><p>تحقق لمعرفة ما إذا كان التطبيق محدثًا أم لا:</p><p> <ol> <li> انتقل إلى متجر Play وتأكد من تحديث التطبيق إلى أحدث إصدار </li> </ol> </p><p><br></p><p>تحقق من اتصالك بالإنترنت:</p><p> <li> إذا كان اتصالك بالإنترنت بطيئًا ، فحاول إعادة الاتصال بشبكة Wi-Fi أو الاتصال بشبكة أخرى. </li> </p><p><br></p><p>اطلب من المشرف التحقق من أجهزتهم واتصال الإنترنت:</p><p> <li> اطلب من المشرف استكشاف الأخطاء وإصلاحها باستخدام الخطوات المذكورة أعلاه </li> </p><p><br></p><p>أخبرنا إذا كنت لا تزال تواجه مشكلات في التحميل:</p><p> <li> أبلغ عن مشكلة عن طريق الاتصال بنا على [email protected]. </li> </p></string> | ||
<string name="faq_answer_7"><p>إذا لم يتم تشغيل الصوت الخاص بك</p><p><br></p><p>تحقق لمعرفة ما إذا كان التطبيق محدثًا أم لا:</p><p> <li>انتقل إلى متجر Play وتأكد من تحديث التطبيق إلى أحدث إصدار</li> </p><p><br></p><p>تحقق من اتصالك بالإنترنت:</p><p> <li>إذا كان اتصالك بالإنترنت بطيئًا ، فحاول إعادة الاتصال بشبكة Wi-Fi أو الاتصال بشبكة أخرى. قد يتسبب الإنترنت البطيء في تحميل الصوت بشكل غير منتظم ، مما يجعل من الصعب تشغيله.</li> </p><p><br></p><p>اطلب من المسؤول التحقق من أجهزتهم واتصال الإنترنت:</p><p> <li>اطلب من المسؤول استكشاف الأخطاء وإصلاحها باستخدام الخطوات المذكورة أعلاه</li> </p><p><br></p><p>أخبرنا إذا كنت لا تزال تواجه مشكلات في التحميل:</p><p> <li>أبلغ عن مشكلة عن طريق الاتصال بنا على [email protected].</li></p></string> | ||
<string name="faq_answer_8"><p>لتنزيل استكشاف:</p><p>1. من الصفحة الرئيسية ، انقر فوق موضوع أو استكشاف.</p><p>2. من صفحة الموضوع هذه ، انقر على علامة التبويب <strong> معلومات </ strong>.</p><p>3. اضغط على <strong> تنزيل الموضوع </ strong>.</p><p>4. اعتمادًا على إعدادات التطبيق ، قد تحتاج إلى موافقة المسؤول أو اتصال Wifi ثابتًا لإكمال التنزيل. إذا لزم الأمر ، بمجرد استيفاء هذه المتطلبات ، يتم تنزيل الموضوع على الجهاز ويمكن استخدامه في وضع عدم الاتصال بواسطة جميع ملفات التعريف. <p></string> | ||
<string name="faq_answer_9"><p> إذا لم تتمكن من العثور على سؤالك أو كنت ترغب في الإبلاغ عن خطأ ، فاتصل بنا على [email protected]. </p></string> | ||
<string name="profile_edit_fragment_test_activity_label">نشاط اختبار جزء تحرير ملف التعريف</string> | ||
<string name="administrator_controls_fragment_test_activity_label">يتحكم المسؤول في نشاط اختبار التجزئة</string> | ||
|
Oops, something went wrong.