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

[Auth] #415 auth instance #452

Merged
merged 23 commits into from
Aug 19, 2022
Merged

[Auth] #415 auth instance #452

merged 23 commits into from
Aug 19, 2022

Conversation

llbartekll
Copy link
Contributor

Description

Add Auth singletone instance that wraps auth client

Resolves #415

How Has This Been Tested?

  • needs to be tested in example app #451

Due Dilligence

  • Breaking change
  • Requires a documentation update

@arein arein added the accepted label Aug 19, 2022
@llbartekll llbartekll changed the title #415 auth instance [Auth] #415 auth instance Aug 19, 2022

public static var instance: AuthClient = {
guard let metadata = Auth.metadata else {
fatalError("Error - you must call Sign.configure(_:) before accessing the shared instance.")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auth.configure(_:)

Sources/Auth/Auth.swift Outdated Show resolved Hide resolved
logger.warn("unexpected request")
return
}
guard let address = address else {return}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@llbartekll llbartekll merged commit 8ddc520 into develop Aug 19, 2022
@llbartekll llbartekll deleted the #415-auth-instance branch August 19, 2022 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants