Clicking the "Skip" button on the validation window causes a hard crash. I'm not super sure if the skip button even needs to exist. The validation process is pretty fast even when doing lots of http header checks. Might be better to force users to click the "Ok" button.

build/Release/ImagrAdmin.app/Contents/MacOS/ImagrAdmin
2016-07-26 09:13:09.678 ImagrAdmin[10885:8849353] User selected path /Users/clburlison/Desktop/min_case_skip.plist
2016-07-26 09:13:09.679 ImagrAdmin[10885:8849353] Need to validate /Users/clburlison/Desktop/min_case_skip.plist
2016-07-26 09:13:09.684 ImagrAdmin[10885:8849353] Initialized ImagrConfigManager.sharedManager with /Users/clburlison/Desktop/min_case_skip.plist
2016-07-26 09:13:09.684 ImagrAdmin[10885:8849353] Updating config view
2016-07-26 09:13:10.359 ImagrAdmin[10885:8849353] Validation task terminated
[1]    10885 illegal hardware instruction  build/Release/ImagrAdmin.app/Contents/MacOS/ImagrAdmin

min_case_skip.txt

New issue checklist

• I have searched existing issues and this is not a duplicate.

Question or Feature Request

There have been a number of new features added to Imagr that have not been added to ImagrAdmin. Currently ImagrAdmin will overwrite and remove the following keys.

• Add support for:
  • autorun_time
  • wait_for_network
  • first_boot_restart

Another thing that need to be done is to create an unknown component feature that will preserve plist details for components that ImagrAdmin does not yet support. This will at least allow users to manually edit their plists for those components .

• I have searched existing issues and this is not a duplicate.

Question or Feature Request

Would it be possible to set a default repo location (or locations...) in the app? Currently having to type out (or paste) the full path to add scripts seems like an opportunity for typos or mistakes...

Thanks for your great work so far!

Peter

Currently ImagrAdmin is left running but in a useless state when the Red X is clicked to close the app.

The really simple solution is master...clburlison:closeOnRedX.

A better option would be to handle this in the MainViewController (I think). Then we should be able to prompt the end user if they want to save when they have modifications. I don't believe you can/should handle that type of task in the AppDelegate which is why I don't recommend the above.

Currently when ImagrAdmin launches you are forced to select an existing config file for modification. We should have the option to create a new config file from scratch.

That might have implementations for the save button. Not actually sure on that part.

New issue checklist

• I have searched existing issues and this is not a duplicate.

General information

• Release Version:
• Reproducible every time? (Yes/No): Yes
• Related issues:

Bug report

Expected behavior

When opening a plist while there is one already open it should replace all data in visible fields with the new plist data.

Actual behavior

Currently if you open a new plist while one is already open the data from both is merged.

Steps to reproduce

1. Open or Create a new config plist.
2. File -> Open
3. Workflows are now merged.

Crash log? Screenshots?

N/A

Can likely remove Format and View completely. File menu items should be useful new, open, save, and close.

New issue checklist

• I have searched existing issues and this is not a duplicate.

General information

• Release Version:
• Reproducible every time? (Yes/No):
• Related issues:

Bug report

Expected behavior

When the US is selected, macOS expected Time Zones are selectable under localization. Eg. America/New_York

Actual behavior

Only US/Eastern (and similar) are shown, hangs on First Boot since imagr script fails to localize.

Steps to reproduce

Set time zone to US/Eastern in the localization step of the workflow.

Crash log? Screenshots?

Question or Feature Request

Bug Report