-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: added ability to handle course errors #18
Conversation
f723215
to
40cf719
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @dixidroid, Moiz approved these changes as we have to open a Modal for purchase that's why we hide that UI part, and the button design is common for whole app. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm supposed to review the IAP flow and found the following issue.
Data is not getting refreshed after the completion of IAP flow when the user doesn't have the course access.
d5a4482
to
5d59994
Compare
- Integrate and parse CourseEnrollmentDetails API - Handle CourseAccess Errors on course Dashboard - Update UI based on CourseAccess Errors. fix:LEARNER-10019
5d59994
to
a888978
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The IAP Flow from the course dashboard screen is working fine for me.
Description:
figma: https://www.figma.com/design/iZ56YMjbRMShCCDxqrqRrR/Open-edX-Mobile-App-All-Screens-v2.2?node-id=10061-77377&t=Gmoit4lBJxrmRE9f-4