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.

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

feat: Add new managed setupassistant payload #5

Closed
wants to merge 2 commits into from

Conversation

clburlison
Copy link

This is a breaking api change since I've modified the old setup assistant to use node['cpe_setupassistant']['once'].

This allows us to use the 'SkipCloudSetup' key that was introduced in 10.10. reference

@erikng
Copy link
Collaborator

erikng commented Jan 22, 2018

I'll test this tomorrow and report back, but everything looks good.

@clburlison
Copy link
Author

Don't merge yet. Since Apple released the following https://developer.apple.com/library/content/featuredarticles/iPhoneConfigurationProfileRef/Introduction/Introduction.html

I'll add these options with comments. Note Apple's docs has the SkipCloudSetup key availably OS version incorrect.

SkipCloudSetup Boolean Optional. If true, skip the Apple ID setup window.Availability: Available in macOS 10.12 and later.
SkipSiriSetup Boolean Optional. If true, skip the Siri setup window.Availability: Available in macOS 10.12 and later.
SkipPrivacySetup Boolean Optional. If true, skip the Privacy consent window.Availability: Available in macOS 10.13.4 and later.
SkipiCloudStorageSetup Boolean Optional. If true, skip the iCloud Storage window.Availability: Available in macOS 10.13.4 and later.

@clburlison
Copy link
Author

I've added the new 10.13.4 keys.

@erikng
Copy link
Collaborator

erikng commented Feb 26, 2018

@clburlison can you add a newline to the attributes file?

@erikng
Copy link
Collaborator

erikng commented Feb 26, 2018

and the resources file.

@erikng
Copy link
Collaborator

erikng commented Feb 26, 2018

There are a few issues I've found with this and I think I ultimately know the issue. I'm going to rewrite it and keep your documentation/key structure/UUIDs.

Thanks for doing the first part.

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.

None yet

2 participants