-
Notifications
You must be signed in to change notification settings - Fork 8
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
[New] Authenticate with OAuth #126
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking great so far!
Shared/Samples/Authenticate with OAuth/AuthenticateWithOAuthView.swift
Outdated
Show resolved
Hide resolved
Shared/Samples/Authenticate with OAuth/AuthenticateWithOAuthView.swift
Outdated
Show resolved
Hide resolved
Shared/Samples/Authenticate with OAuth/AuthenticateWithOAuthView.swift
Outdated
Show resolved
Hide resolved
Co-authored-by: Vivian Quach <viv10382@esri.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If I click on cancel when the login alert shows, it gives this warning
I don't think it matters, might just be a false alarm from UIKit.
2023-03-17 16:06:08.477191-0700 Samples[2855:10999341]
[Warning] Attempting to load the view of a view controller while it is deallocating is not allowed
and may result in undefined behavior (<SFAuthenticationViewController: 0x7fb94a09b200>)
Shared/Samples/Authenticate with OAuth/AuthenticateWithOAuthView.swift
Outdated
Show resolved
Hide resolved
Shared/Samples/Authenticate with OAuth/AuthenticateWithOAuthView.swift
Outdated
Show resolved
Hide resolved
Shared/Samples/Authenticate with OAuth/AuthenticateWithOAuthView.swift
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will approve after Esri/arcgis-maps-sdk-swift-toolkit#276 is merged.
Shared/Samples/Authenticate with OAuth/AuthenticateWithOAuthView.swift
Outdated
Show resolved
Hide resolved
Shared/Samples/Authenticate with OAuth/AuthenticateWithOAuthView.swift
Outdated
Show resolved
Hide resolved
Shared/Samples/Authenticate with OAuth/AuthenticateWithOAuthView.swift
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just tested. Looks good!
Thanks @yo1995 and @vquach2404! |
Description
This PR implements
Authenticate with OAuth
inCloud and portal
category.URL to README: README
Linked Issue(s)
serenity/issues/1406
How To Test
Screenshots