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

Add support for richie site factory #4

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

BbrSofiane
Copy link

@BbrSofiane BbrSofiane commented Aug 19, 2022

Problem

Tutor users can deploy and use Richie using this plugin. However, it seems that the only way to deploy a customised instance of Richie is to use a fork of Richie.

From openfun documentation, the recommended path to customise an instance is to use a Richie site factory. This is not supported by this plugin.

Thread

Suggested solution

This PR is my first attempt at providing support for a Richie site factory.

My goal is to implement a solution that keeps the user experience simple. Ideally, users just need to provide a RICHIE_FACTORY_REPOSITORY and RICHIE_FACTORY_SITE to deploy their customise instance.

@BbrSofiane BbrSofiane force-pushed the bbrsofiane/support-site-factory branch 3 times, most recently from 8c3b7df to fd29bcd Compare August 19, 2022 18:23
@BbrSofiane
Copy link
Author

Current implementation

Use needs to set RICHIE_FACTORY_REPOSITORY, RICHIE_FACTORY_SITE and RICHIE_FACTORY_VERSION.

TODO

  • Get feedback on the implementation
  • Update init scripts
  • Update k8s deployment
  • Use redis service for caches
  • Update Readme

@BbrSofiane BbrSofiane force-pushed the bbrsofiane/support-site-factory branch 2 times, most recently from 95f1486 to 4f16528 Compare August 23, 2022 12:00
@BbrSofiane
Copy link
Author

Use Upgrade instructions from Richie.

@BbrSofiane BbrSofiane force-pushed the bbrsofiane/support-site-factory branch from 5ea228a to afec706 Compare September 30, 2022 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant