Navigation Menu

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

Add new pppc keys #81

Merged
merged 46 commits into from Oct 22, 2020
Merged

Add new pppc keys #81

merged 46 commits into from Oct 22, 2020

Conversation

watkyn
Copy link
Contributor

@watkyn watkyn commented Oct 14, 2020

  • There is now an option for using the new Big Sur Authorization Key for the profile.
  • Selecting one of the new keys will automatically prompt the user with an alert that it is only Big Sur compatible
  • I also updated the UI to move the import button down by the save button
  • Added some more unit tests

watkyn and others added 25 commits October 1, 2020 13:39
…he new Authorization key, warning them it is only Big Sur and above
…and an incomplete UI implementation of the NSSwitch for big sur.
…s are going away from big sur mode and might lose some data.
…ill one annoying one not resolved about a fixed width, and also some content notices on the storyboard.
@watkyn watkyn requested review from hisaac and pmoprhy October 14, 2020 17:37
@watkyn
Copy link
Contributor Author

watkyn commented Oct 14, 2020

I'll have to look into why I didn't get the SwiftLint error locally... version maybe.

@watkyn
Copy link
Contributor Author

watkyn commented Oct 14, 2020

I'll have to look into why I didn't get the SwiftLint error locally... version maybe.
yep, needed to update locally

PPPC Utility.xcodeproj/project.pbxproj Show resolved Hide resolved
PPPC UtilityTests/ModelTests/ModelTests.swift Outdated Show resolved Hide resolved
Source/View Controllers/TCCProfileViewController.swift Outdated Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
@watkyn watkyn linked an issue Oct 14, 2020 that may be closed by this pull request
@watkyn watkyn requested a review from macblazer October 15, 2020 15:04
Copy link
Contributor

@hisaac hisaac left a comment

Choose a reason for hiding this comment

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

No glaring issues I could see, just a couple of formatting comments.

I hope we can sort out the signing issue. If we can't figure it out, it'd be good to call it out in the readme somewhere to let people know how to work around it, especially if we want less experienced developers to work on the app.

Source/Model/Model.swift Outdated Show resolved Hide resolved
case .on:
if model.usingLegacyAllowKey && showAlert {
let message = """
Enabling Big Sur Compatibility will require the profile to be installed on macOS versions Big Sur (11.0) or greater.
Copy link
Contributor

Choose a reason for hiding this comment

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

Think it'd be worth using 10.16 here instead? Or 10.16/11.0?

Suggested change
Enabling Big Sur Compatibility will require the profile to be installed on macOS versions Big Sur (11.0) or greater.
Enabling Big Sur Compatibility will require the profile to be installed on macOS versions Big Sur (10.16) or greater.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, I'm not sure what to do there. I'm not a fan of Apple's conditional versioning for Big Sur.

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we could just omit the number and say "on macOS Big Sur or later".

@hisaac
Copy link
Contributor

hisaac commented Oct 16, 2020

So here's an interesting development: I just switched to my personal Mac, rolled back to the version before you disabled the hardened runtime, and the unit tests ran just fine.

I also found another, non-Jamf Xcode project that was having the same issues. Tests hang indefinitely on my work Mac, but run just fine on my personal Mac. 🤔

@watkyn watkyn merged commit 9878cc2 into master Oct 22, 2020
@watkyn watkyn deleted the add-new-pppc-keys branch October 22, 2020 14:34
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.

New PPPC Keys available in Big Sur beta 9
4 participants