Skip to content
This repository has been archived by the owner on May 6, 2024. It is now read-only.

feat: Learner feedback collection from the mobile app #1845

Merged
merged 1 commit into from
Feb 15, 2024

Conversation

farhan-arshad-dev
Copy link
Contributor

Description

LEARNER-9834

  • Learner feedback collection from the mobile app.
  • A "Submit Feedback" button should be added to the help section of the profile screen above the “Email the support team” in the edX mobile app.
  • The button should feature an icon indicating that it leads to an external link.
  • When the "Submit Feedback" button is tapped, it should open an external link, launching a Google Form in the user's default browser.
  • Trigger event on Submit Feedback button clicked.

Config was added in Endpoint:
/api/mobile/{api_version}/users/{username}/course_enrollments

- A "Submit Feedback" button should be added to the help section of the profile screen above the “Email the support team” in the edX mobile app.
- The button should feature an icon indicating that it leads to an external link.
- When the "Submit Feedback" button is tapped, it should open an external link, launching a Google Form in the user's default browser.
- Trigger event on Submit Feedback button clicked

fixes: LEARNER-9834
@farhan-arshad-dev farhan-arshad-dev requested review from omerhabib26 and HamzaIsrar12 and removed request for omerhabib26 February 14, 2024 16:35
Copy link

codecov bot commented Feb 14, 2024

Codecov Report

Attention: 14 lines in your changes are missing coverage. Please review.

Comparison is base (a6b4bac) 1.07% compared to head (8557a0c) 1.07%.

Files Patch % Lines
...c/main/java/org/edx/mobile/view/AccountFragment.kt 0.00% 10 Missing ⚠️
...rc/main/java/org/edx/mobile/model/api/AppConfig.kt 0.00% 3 Missing ⚠️
.../java/org/edx/mobile/module/prefs/FeaturesPrefs.kt 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##             master   #1845      +/-   ##
===========================================
- Coverage      1.07%   1.07%   -0.01%     
  Complexity      137     137              
===========================================
  Files           538     538              
  Lines         26313   26326      +13     
  Branches       3391    3395       +4     
===========================================
  Hits            284     284              
- Misses        26002   26015      +13     
  Partials         27      27              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@omerhabib26 omerhabib26 left a comment

Choose a reason for hiding this comment

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

LGTM 🏎️

@farhan-arshad-dev farhan-arshad-dev merged commit 2b52504 into master Feb 15, 2024
4 checks passed
@farhan-arshad-dev farhan-arshad-dev deleted the farhan_ar/LEARNER-9834 branch February 15, 2024 06:25
@farhan-arshad-dev farhan-arshad-dev restored the farhan_ar/LEARNER-9834 branch February 15, 2024 06:25
@farhan-arshad-dev farhan-arshad-dev deleted the farhan_ar/LEARNER-9834 branch February 15, 2024 06:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants