openedx / platform-roadmap

Tracking the maintenance, enhancement, and advancement of the Open edX project.
11 stars 0 forks source link

[Product Pull Request] feat: Support web-based LMS OAuth #345

Open jmakowski1123 opened 5 months ago

jmakowski1123 commented 5 months ago

Master ticket for the project.

Child PRs:

github-actions[bot] commented 5 months ago

Thanks for your submission, @openedx/open-edx-project-managers will review shortly.

itsjeyd commented 4 months ago

@xitij2000 @Kelketek If I understand correctly (?), https://github.com/openedx/openedx-app-android/pull/190 is closely related to https://github.com/openedx/openedx-app-ios/pull/65, the only PR listed as a child of this roadmap item so far.

Would it make sense to add https://github.com/openedx/openedx-app-android/pull/190 to the list above?

CC @jmakowski1123

xitij2000 commented 4 months ago

@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?

CC @jmakowski1123

Yes, I think that is the latest version of the "login-via-LMS" work.

mphilbrick211 commented 1 month ago

Hi all, just looking for an update here (@jmakowski1123)

Kelketek commented 1 month ago

@mphilbrick211 The sponsoring organization decided not to move this forward to completion, unfortunately, so it's stuck.

itsjeyd commented 1 month ago

@Kelketek @mphilbrick211 For https://github.com/openedx/openedx-app-android/pull/190 specifically, I think we might actually have a way forward.

@xitij2000 Does the new PR that you created a couple days ago (https://github.com/openedx/openedx-app-android/pull/371) fully replace that PR?

xitij2000 commented 1 month ago

@itsjeyd Yes, I should perhaps have updated the same PR, however, this one is the same code rebased to the latest main branch.

itsjeyd commented 1 month ago

@xitij2000 OK, thanks for confirming 👍

@mphilbrick211 @jmakowski1123 Could we get https://github.com/openedx/openedx-app-android/pull/371 added to the list of child PRs for this feature ticket in the description above?

CC @Kelketek