-
Notifications
You must be signed in to change notification settings - Fork 1
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
[Product Pull Request] feat: Support web-based LMS OAuth #345
Comments
Thanks for your submission, @openedx/open-edx-project-managers will review shortly. |
@xitij2000 @Kelketek If I understand correctly (?), openedx/openedx-app-android#190 is closely related to openedx/openedx-app-ios#65, the only PR listed as a child of this roadmap item so far. Would it make sense to add openedx/openedx-app-android#190 to the list above? |
Yes, I think that is the latest version of the "login-via-LMS" work. |
Hi all, just looking for an update here (@jmakowski1123) |
@mphilbrick211 The sponsoring organization decided not to move this forward to completion, unfortunately, so it's stuck. |
@Kelketek @mphilbrick211 For openedx/openedx-app-android#190 specifically, I think we might actually have a way forward. @xitij2000 Does the new PR that you created a couple days ago (openedx/openedx-app-android#371) fully replace that PR? |
@itsjeyd Yes, I should perhaps have updated the same PR, however, this one is the same code rebased to the latest main branch. |
@xitij2000 OK, thanks for confirming 👍 @mphilbrick211 @jmakowski1123 Could we get openedx/openedx-app-android#371 added to the list of child PRs for this feature ticket in the description above? CC @Kelketek |
Is there an update on this? |
@mphilbrick211 The Android feature has apparently been merged, but I haven't gotten confirmation on it for the iOS app. |
@jmakowski1123 Did you mean to close this ticket? Android got this feature but I don't think iOS did. The PR on this ticket hasn't been merged either. |
Master ticket for the project.
Child PRs:
The text was updated successfully, but these errors were encountered: