Skip to content
This repository has been archived by the owner on May 2, 2024. It is now read-only.

Rewrite the tutor-mfe plugin to use the V1 API #59

Closed
kdmccormick opened this issue Apr 14, 2022 · 2 comments
Closed

Rewrite the tutor-mfe plugin to use the V1 API #59

kdmccormick opened this issue Apr 14, 2022 · 2 comments
Assignees
Labels
bug Report of or fix for something that isn't working as intended

Comments

@kdmccormick
Copy link
Collaborator

kdmccormick commented Apr 14, 2022

Context

Once #32 is finished, we'll eventually need to rewrite the official plugins. It'll be good to have at least one plugin rewritten in time for the Lisbon conference. Let's try to do tutor-mfe, as there is a specific bugfix we could make to it once it's using the V1 API: https://discuss.overhang.io/t/why-is-the-mfe-image-not-pre-built/2260/6?u=kdmccormick

Acceptance

Do it.

Integrate this hack into the rewrite: https://discuss.overhang.io/t/why-is-the-mfe-image-not-pre-built/2260/6?u=kdmccormick

@kdmccormick kdmccormick added bug Report of or fix for something that isn't working as intended refactor labels Apr 14, 2022
regisb added a commit to overhangio/tutor-mfe that referenced this issue Apr 15, 2022
We upgrade this plugin to make use of the v1 plugin API. As a consequence, we
are able to condition image pull/push based on whether users have modified the
Docker image tag.

Inspiration from this neat trick: https://discuss.overhang.io/t/why-is-the-mfe-image-not-pre-built/2260/6

See: openedx-unsupported/wg-developer-experience#59
@regisb
Copy link

regisb commented Apr 15, 2022

Here's the PR: overhangio/tutor-mfe#44

regisb added a commit to overhangio/tutor-mfe that referenced this issue Apr 15, 2022
We upgrade this plugin to make use of the v1 plugin API. As a consequence, we
are able to condition image pull/push based on whether users have modified the
Docker image tag.

Inspiration from this neat trick: https://discuss.overhang.io/t/why-is-the-mfe-image-not-pre-built/2260/6

See: openedx-unsupported/wg-developer-experience#59
@regisb regisb self-assigned this Apr 15, 2022
@regisb regisb moved this from Groomed to In Progress in Tutor DevEnv Adoption (OLD BOARD) Apr 15, 2022
@kdmccormick kdmccormick moved this from In Progress to In Review in Tutor DevEnv Adoption (OLD BOARD) Apr 18, 2022
regisb added a commit to overhangio/tutor-mfe that referenced this issue Apr 19, 2022
We upgrade this plugin to make use of the v1 plugin API. As a consequence, we
are able to condition image pull/push based on whether users have modified the
Docker image tag.

Inspiration from this neat trick: https://discuss.overhang.io/t/why-is-the-mfe-image-not-pre-built/2260/6

See: openedx-unsupported/wg-developer-experience#59
regisb added a commit to overhangio/tutor-mfe that referenced this issue Apr 19, 2022
We upgrade this plugin to make use of the v1 plugin API. As a consequence, we
are able to condition image pull/push based on whether users have modified the
Docker image tag.

Inspiration from this neat trick: https://discuss.overhang.io/t/why-is-the-mfe-image-not-pre-built/2260/6

See: openedx-unsupported/wg-developer-experience#59
regisb added a commit to overhangio/tutor-mfe that referenced this issue Apr 19, 2022
We upgrade this plugin to make use of the v1 plugin API. As a consequence, we
are able to condition image pull/push based on whether users have modified the
Docker image tag.

Inspiration from this neat trick: https://discuss.overhang.io/t/why-is-the-mfe-image-not-pre-built/2260/6

See: openedx-unsupported/wg-developer-experience#59
@regisb regisb moved this from In Review to Closed in Tutor DevEnv Adoption (OLD BOARD) Apr 19, 2022
@kdmccormick
Copy link
Collaborator Author

was closed last week by overhangio/tutor-mfe#44

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Report of or fix for something that isn't working as intended
Projects
None yet
Development

No branches or pull requests

2 participants