Skip to content

(Fix) don't automatically enable courseware discovery for devstack - #198

Merged
bryanlandia merged 1 commit into
appsembler/ficus/developfrom
appsembler/ficus/hotfix/devstack-obey-discovery-settings-rework
Jan 11, 2018
Merged

(Fix) don't automatically enable courseware discovery for devstack#198
bryanlandia merged 1 commit into
appsembler/ficus/developfrom
appsembler/ficus/hotfix/devstack-obey-discovery-settings-rework

Conversation

@bryanlandia

@bryanlandia bryanlandia commented Jan 9, 2018

Copy link
Copy Markdown

This is a fix to a previous PR which had an error (#191).

Closes #197

@bryanlandia
bryanlandia requested a review from OmarIthawi January 9, 2018 19:09

@OmarIthawi OmarIthawi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's much better, thanks @bryanlandia!

Please go ahead and merge it if it works on your machine.

@bryanlandia
bryanlandia force-pushed the appsembler/ficus/hotfix/devstack-obey-discovery-settings-rework branch from 26694ed to 7343327 Compare January 11, 2018 01:16
@bryanlandia
bryanlandia merged commit 8c704f9 into appsembler/ficus/develop Jan 11, 2018
@bryanlandia
bryanlandia deleted the appsembler/ficus/hotfix/devstack-obey-discovery-settings-rework branch January 11, 2018 01:17
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.

2 participants