-
Notifications
You must be signed in to change notification settings - Fork 290
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
Missing conf folder prevent install #1906
Milestone
Comments
canstudios-louisem
added a commit
that referenced
this issue
Mar 14, 2018
taylortom
added a commit
that referenced
this issue
Mar 14, 2018
Add check for conf directory. fixes Issue/#1906
taylortom
added a commit
that referenced
this issue
Mar 14, 2018
* develop: Require modules before plugins Change to ensureDirSync Fixes #1906 Allow MongoDB authentication in install script Fix modules define Fix globalData Save isVisible and isHidden on blocks and components Fix context menu for page Remove config-sample.json Allow upgrade if missing frameworkRepository or authoringToolRepository Catch errors and return Fix route error logging fixed indenting update mailer to support custom connection url prompt user if custom connection url is used and dynamically build prompt also fixes issue with config options not added to config.json update config sample to include new config options Add instruction to component schema # Conflicts: # frontend/src/core/app.js
canstudios-louisem
added
S: merged
Completed, reviewed, and merged issues
and removed
S: awaiting-review
Completed issues waiting on reviews
labels
Mar 14, 2018
taylortom
added a commit
that referenced
this issue
Mar 21, 2018
* develop: (80 commits) update Check if user is course owner Require modules before plugins Change to ensureDirSync refactor to set scrollTop after all animations are completed hook into transition end event instead of post render Fixes #1906 Allow MongoDB authentication in install script Fix modules define Fix globalData Make sure we load the plugin Fix issue with merge Save isVisible and isHidden on blocks and components Fix context menu for page Move plugin folder to modules Refactor for readability Update comments Fix plugin code broken in refactor Persist sort prefs indefinitely Remove config-sample.json ... # Conflicts: # frontend/src/core/helpers.js
taylortom
added a commit
that referenced
this issue
Mar 22, 2018
* develop: (80 commits) update Check if user is course owner Require modules before plugins Change to ensureDirSync refactor to set scrollTop after all animations are completed hook into transition end event instead of post render Fixes #1906 Allow MongoDB authentication in install script Fix modules define Fix globalData Make sure we load the plugin Fix issue with merge Save isVisible and isHidden on blocks and components Fix context menu for page Move plugin folder to modules Refactor for readability Update comments Fix plugin code broken in refactor Persist sort prefs indefinitely Remove config-sample.json ... # Conflicts: # frontend/src/modules/editor/global/views/editorView.js
taylortom
added a commit
that referenced
this issue
Apr 5, 2018
* develop: (111 commits) update archive dependecy Fixes: #1928 Fix and refactor helper update Check if user is course owner Require modules before plugins Change to ensureDirSync refactor to set scrollTop after all animations are completed hook into transition end event instead of post render Fixes #1906 Allow MongoDB authentication in install script Fix modules define Fix globalData Make sure we load the plugin Fix issue with merge Save isVisible and isHidden on blocks and components Fix context menu for page Move plugin folder to modules Refactor for readability Update comments ... # Conflicts: # frontend/src/core/helpers.js # frontend/src/modules/assetManagement/index.js # frontend/src/modules/editor/contentObject/index.js # frontend/src/modules/editor/course/index.js
taylortom
added a commit
that referenced
this issue
Apr 9, 2018
* develop: (94 commits) update archive dependecy Fixes: #1928 Fix and refactor helper update Check if user is course owner Require modules before plugins Change to ensureDirSync refactor to set scrollTop after all animations are completed hook into transition end event instead of post render Fixes #1906 Allow MongoDB authentication in install script Fix modules define Fix globalData Make sure we load the plugin Fix issue with merge Save isVisible and isHidden on blocks and components Fix context menu for page Move plugin folder to modules Refactor for readability Update comments ... # Conflicts: # plugins/output/adapt/index.js
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Expected Behaviour
Able to do a new install of the authoring tool.
Actual Behaviour
Install fails with the following error:
Most likely due to this PR. Install should ensure conf directory is present before writing config.json.
Steps to Reproduce
Versions
The text was updated successfully, but these errors were encountered: