Ecommerce is no longer optional in edx_sandbox.yml #3585

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@nedbat
Member
nedbat commented Jan 3, 2017

Configuration Pull Request

Make sure that the following steps are done before merging

  • @devops team member has commented with 👍
  • are you adding any new default values that need to be overriden when this goes live?
    • Open a ticket (DEVOPS) to make sure that they have been added to secure vars.
    • Add an entry to the CHANGELOG.
@nedbat nedbat Ecommerce is no longer optional in edx_sandbox.yml
20b14ed
@nedbat
Member
nedbat commented Jan 3, 2017

@antoviaque @bradenmacdonald I'm told that I am the only edXer using edx_sandbox.yml any more, but that OpenCraft still does. What do you think of removing the optionalness of ecommerce? I can leave it, and set the variable instead if you still want it.

@bradenmacdonald
Member

@nedbat We would appreciate having it around for a couple more months. At some point this year we will completely transition to the edx-stateless.yml playbook, so then it won't matter to us.

@nedbat
Member
nedbat commented Jan 3, 2017

OK, no problem.

@nedbat nedbat closed this Jan 3, 2017
@nedbat nedbat deleted the ned/install-ecommerce branch Jan 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment