-
Notifications
You must be signed in to change notification settings - Fork 11
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
Fix macOS can't connect because of an error 400 #1
Conversation
Thanks for the PR 👍. We'll be sure to include this in a future release |
Hey @glhaynes, before we pull this in - Would you mind taking a look at our (newly added) contributing guidelines? Specifically, would you mind reading through and agreeing to the Individual Contributor License Agreement (CLA). We really appreciate the contribution and the testing you've done for this project in its early stages! |
Hi, @atreat! Excited for y'all to include this little fix. I get this when I click on both of those links, though: You need permission |
@glhaynes That's on me, I accidentally pasted the same link twice. The contributing guidelines link should be updated. It is also found in the repo at Let me know if you have chance to look through these. Thank you! |
Done. Thanks! |
We pulled this in! Going to close this as it is now on embrace-apple-sdk/Sources/EmbraceObjCUtils/source/EMBDevice.m Lines 335 to 343 in 00109f3
We liked this implementation and preferred to remove the |
Oh, great point, it didn't cross my mind that those would be equally available on iOS! Happy to help, excited to see this in beta as we're looking forward to adopting it. |
Return the correct value for
EMBDevice.operatingSystemVersion
when on macOS. Fixes an issue where macOS Embrace can't connect because of an error 400: