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

Finalize Studio remote browsing integration #10280

Closed
rtibbles opened this issue Mar 20, 2023 · 1 comment · Fixed by #10450
Closed

Finalize Studio remote browsing integration #10280

rtibbles opened this issue Mar 20, 2023 · 1 comment · Fixed by #10450
Assignees
Labels
APP: Learn Re: Learn App (content, quizzes, lessons, etc.) DEV: frontend P0 - critical Priority: Release blocker or regression

Comments

@rtibbles
Copy link
Member

Observed behavior

#10227 was merged with some fairly involved special case logic for Studio that uses the existing RemoteChannelsEndpoint.

Now that the implementation of the public APIs on Studio have been finalized and merged to unstable, we can now do further testing of this flow using unstable as our upstream Studio, and remove most of th

Expected behavior

With the new API endpoints in place, we should be able to check Studio's info endpoint, and confirm that it is indeed upgraded to a version equivalent to Kolibri 0.16.0 (using the minVersion composable logic), and then display it if so.

We should then be able to browse the device exactly like any other, with current device info that sets the baseurl to the set Studio URL in Kolibri configuration.

To test with unstable we can override this configuration by setting this environment variable:

KOLIBRI_CENTRAL_CONTENT_BASE_URL=https://unstable.studio.learningequality.org/
@rtibbles rtibbles added P0 - critical Priority: Release blocker or regression APP: Learn Re: Learn App (content, quizzes, lessons, etc.) DEV: frontend labels Mar 20, 2023
@bjester
Copy link
Member

bjester commented Apr 11, 2023

Added #10431

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
APP: Learn Re: Learn App (content, quizzes, lessons, etc.) DEV: frontend P0 - critical Priority: Release blocker or regression
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants