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

feature: Ported the sample project to MacOS swift with some minor mod… #61

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

feature: Ported the sample project to MacOS swift with some minor mod… #61

wants to merge 2 commits into from

Conversation

grantkemp
Copy link

As mentioned in #60
Ported the sample project to MacOS swift with some minor mod…ifications ( as little as possible) to keep things comparable.

  • Added some simplified documentation to Readme and into the project adjacent to the relevant variables so that its easier to understand whats needed.
  • Added in docs for some missing steps that bamboozled me and lost me time.
  • Moved Scopes up to a variable and showed other users where to access them. This was really helpful to me to try get going with google services.

Pending Tasks for this item ( if this is going to be used)

  • Fixing up the left over todos- I didn't update them as they weren't relevant to my use case. If you are going to use them in the project, then can tidy them up.
  • Update syntax to match Swift current Standards
  • Validate approach is correct.

…ifications ( as little as possible) to keep things comparable.

- Added some simplified documentation to Readme and into the project adjacent to the relevant variables so that its easier to understand whats needed.
- Added in docs for  some missing steps that bamboozled me and lost me time.
- Moved Scopes up to a variable and showed other users where to access them.  This was really helpful to me to try get going with google services.
@OxenBoxen
Copy link

Hey @grantkemp , I too am writing a Swift MacOS app at the moment, and am using this library. I am also experiencing problems with Chome - I'm wondering how can I use or fork your updates? It does not appear your updates are merged in master yet. Thanks for the help!

@grantkemp
Copy link
Author

Hiya - Go Ahead. I would be delighted.
I have been using the library a lot now and have got my head around it. If you need a hand - ping me here or on twitter @ukdatageek .

@google google deleted a comment from alateeq99 May 31, 2018
@google google deleted a comment from alateeq99 May 31, 2018
@expedited
Copy link

Hi there, maybe this is coming a little late but I just found this swift port yesterday, thanks for this. It seemed like a nightmare to me reading through the dense obj c examples and trying to reimagine it as swift code. To be clear, I'm trying to create a MacOS Cocoa app, not an iOS app. It seems like the code here is the authorization code for iOS, or is actually for MacOS?

I tried copy pasting your code into an example yesterday, and I was met with an invalid request URI. I read through the original documentation and it looks like for MacOS we want to create a local HTTP server to listen on, but from what I can see in this example it looks more like the iOS authorization. Did I just get confused here or is this example actually aimed at iOS swift?

Thanks!

@grantkemp
Copy link
Author

@expedited I built it a while back - it seemed to work for me - but there have been a lot of changes since! I might do another project with it - if so I will update !

@elsesiy
Copy link

elsesiy commented Apr 14, 2019

@expedited You can also check out my wrapper for this which does work with MacOS and iOS. I added MacOS support a while back as @grantkemp was asking for it.

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.

5 participants