-
Notifications
You must be signed in to change notification settings - Fork 379
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
API design: Get unlock PIN immediately after locking macOS host #19671
Conversation
Hey @rachaelshaw do you remember why we decided to include the I omitted Also, I updated the status code from |
@noahtalerman I don't remember why we included it, and can't find any conversation about it in the original PR. Doesn't seem necessary. I'm feeling torn between being consistent and simplifying the response; probably makes sense to simplify though.
Shouldn't be since it's still a 2xx status code |
@noahtalerman converted to draft PR since this isn't implemented yet |
Closing this draft PR and opening a new one so we don't affect the PR open time KPI. |
UPDATE: Here's the new PR:
Proposed API changes for the "Get unlock PIN immediately after locking macOS host" story (#19545)