Skip to content

Resolve "Production ready"

Ryan requested to merge feature/#56-production-ready into alpha

Summary

Before releasing version 1, make everything ready for production.

Closes #56 (closed)

Changes

  • Added Makefile for easy access.
  • Added all ENVs to docker-compose.yml.
  • Added Getting Started within README.md.
  • Added using Gitlab's CI/CD masked variables within production.
  • Fixed Docker containers always restarting.
  • Removed user_navigation option inside LTI XMLs.
Inside Authentication/
  • Fixed usage of ENV Consumer and Client Keys & Secrets.
Inside Public/
  • Added initial check if accessed within their own portfolio.
  • Refactored LTI setup launch code.
Inside Learning/
  • Added ability to check if my/portoflio.
  • Added quick checks to environmental variables.
  • Security Don't use Teacher tokens.

Breaking changes

  • Teacher Tokens are dephrecated
  • No longer LTI app inside user settings navigation bar

Before merge

  • Changed code has been reviewed
  • Tested to build, start and is usable
Edited by Ryan

Merge request reports