appsembler / configuration

a simple, but flexible, way for anyone to stand up an instance of the edX platform that is fully configured and ready-to-go
GNU Affero General Public License v3.0
14 stars 13 forks source link

Remove some of our changes #5

Open tkeemon opened 8 years ago

tkeemon commented 8 years ago

These two changes:

can be removed from our version of the fork if we do the following in each customer's server-vars.yml file:


---
EDXAPP_APPSEMBLER_FEATURES: 
  FEATURE_1: value
  FEATURE_2: value

EDXAPP_ENV_EXTRA:
  APPSEMBLER_FEATURES: "{{ EDXAPP_APPSEMBLER_FEATURES }}"
natea commented 8 years ago

@tkeemon agreed that let's try to remove as many customizations from the playbooks as possible, and encapsulate them in server-vars.yml so they're all in one place.

OmarIthawi commented 3 years ago

Thanks TJ for the note. It's 2021 but not too late to address this tech debt. My tentative plan is to remove APPSEMBLER_FEATURES['ENABLE_XYZ'] in favor of settings.FEATUES['TAHOE_ENABLE_XYZ'].