Skip to content

Signin highos #150

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

Closed
wants to merge 13 commits into from
Closed

Signin highos #150

wants to merge 13 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Jul 1, 2018

No description provided.

@ldgmini
Copy link

ldgmini commented Jul 1, 2018

1 Error
🚫 📝 Please provide a summary in the Pull Request description
1 Warning
⚠️ 😵 Big PR

Generated by 🚫 Danger

@phatblat
Copy link
Member

phatblat commented Jul 1, 2018

@PatYo9 the signin-highos branch is not ready to be merged. The main issue I've been hoping to resolve is the signin functionality. The method accountService.signIn(with: context) worked in older versions of macOS, but was removed in High Sierra (10.13) and I have not been able to find an alternative.

Since this is part of a private framework, it is not supported by Apple, which limits our options.

@ghost
Copy link
Author

ghost commented Jul 1, 2018 via email

@todo
Copy link

todo bot commented Aug 11, 2018

Handle failed AppleID lookup

// TODO: Handle failed AppleID lookup
print("No account found for username: \(username)")
}
group.leave()
}


This comment was generated by todo based on a TODO comment in 76b1098 in #150. cc @mas-cli.

@phatblat phatblat closed this Aug 11, 2018
@todo todo bot mentioned this pull request Aug 11, 2018
@phatblat
Copy link
Member

phatblat commented Aug 11, 2018

Superceded by #162

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.

2 participants