Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Install script doesn't take into account all config.json values #1828

Closed
taylortom opened this issue Dec 19, 2017 · 1 comment
Closed

Install script doesn't take into account all config.json values #1828

taylortom opened this issue Dec 19, 2017 · 1 comment
Labels
T: bug

Comments

@taylortom
Copy link
Member

Expected Behaviour

Script should use all values in config.json, removing the need to input these values again.

Actual Behaviour

Script prompts the user for inputs from the inputData.features inputs onwards.

Steps to Reproduce

  1. Run through install script, choosing y when asked to use config.json values

Versions

  • Authoring Tool Version: 0.4.0
@taylortom taylortom added this to the 0.4.1 Bugfixes milestone Dec 19, 2017
@taylortom taylortom mentioned this issue Dec 19, 2017
Merged
@canstudios-louisem
Copy link
Contributor

How important is this for 0.4.1 or can it be pushed back to 0.5?

@taylortom taylortom modified the milestones: 0.4.1 Bugfixes, 0.5.0 Jan 9, 2018
@dancgray dancgray mentioned this issue Jan 16, 2018
Closed
Annoraaq pushed a commit to Annoraaq/adapt_authoring that referenced this issue Feb 21, 2018
@taylortom @Annoraaq
Fix saving of config to file
eceb609 
Fixes adaptlearning#1828
@taylortom taylortom removed this from the Suggested Next milestone Mar 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
T: bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@taylortom @canstudios-louisem