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

Add signup api #78

Merged
merged 1 commit into from Aug 22, 2018
Merged

Add signup api #78

merged 1 commit into from Aug 22, 2018

Conversation

mederic-p
Copy link
Contributor

Closes #77

Overview

This PR adds the new client signup API.

Changes

  • Add SignupParams
  • Add signup endpoint

Usage

The new endpoint can be used as follow:

let credentials = ClientCredential(apiKey: "api_key")
let config = ClientConfiguration(baseURL: "http://localhost:4000", credentials: credentials)
let client = HTTPClientAPI(config: config)
let params = SignupParams(email: "email@example.com", password: "password", passwordConfirmation: "password")
client.signup(withParams: params) { (result) in
    switch result {
    case .success: break
    case let .fail(error: error): break
    }
}

@ghost ghost added the s2/wip 🚧 This is still being worked on label Aug 22, 2018
@mederic-p mederic-p changed the base branch from master to 75-refactor-http-clients August 22, 2018 06:29
@mederic-p mederic-p added kind/enhancement 🚀 and removed s2/wip 🚧 This is still being worked on labels Aug 22, 2018
@ghost ghost added the s2/wip 🚧 This is still being worked on label Aug 22, 2018
@mederic-p mederic-p changed the base branch from 75-refactor-http-clients to master August 22, 2018 07:23
@mederic-p mederic-p merged commit 45deb0c into master Aug 22, 2018
@ghost ghost removed the s2/wip 🚧 This is still being worked on label Aug 22, 2018
@mederic-p mederic-p deleted the 77-add-signup-api branch August 22, 2018 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants