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

[iOS] Issue for PrimaryEnrollment data getting #491

Closed
rnr opened this issue Jul 11, 2024 · 0 comments
Closed

[iOS] Issue for PrimaryEnrollment data getting #491

rnr opened this issue Jul 11, 2024 · 0 comments
Assignees

Comments

@rnr
Copy link
Contributor

rnr commented Jul 11, 2024

The current iOS implementation fails when the primary rate has a non-null audit_access_expires parameter in the data, because the decoder cannot directly convert a String Date to a Data object.
An error occurs when doing so:
typeMismatch(Swift.Double, Swift.DecodingError.Context(codingPath: [CodingKeys(stringValue: "primary", intValue: nil), CodingKeys(stringValue: "audit_access_expires", intValue: nil)], debugDescription: "Expected to decode Double but found a string instead.", underlyingError: nil))
and all the courses to view in the Gallery Dashboard are not displayed

Expected behaviour:
All courses and the primary course should be displayed in the Gallery Dashboard whether audit_access_expires is set or not

@rnr rnr changed the title [iOS] Small Fix for PrimaryEnrollment data getting [iOS] Issue for PrimaryEnrollment data getting Jul 11, 2024
@rnr rnr self-assigned this Jul 11, 2024
@rnr rnr moved this to In Code Review / Final Acceptance in Open edX Mobile Roadmap Jul 11, 2024
@rnr rnr linked a pull request Jul 11, 2024 that will close this issue
@rnr rnr closed this as completed Jul 19, 2024
@github-project-automation github-project-automation bot moved this from In Code Review / Final Acceptance to Done in Open edX Mobile Roadmap Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant