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 methods #19

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

iDevelopThings
Copy link

Found my self adding these changes into my project, so decided I'd update this PR with them instead ^^

Mainly signin/signup will return a token if you sign up a user via a scope for example

I'm also not sure if this will add commits to #17 or create a new PR, either way it includes those changes too.

Keitio and others added 4 commits September 23, 2022 20:25
…e and provides ability to define scope, ns, db etc
This will allow us to do individual tests without re-creating db each time

Wrote tests for `SignupUser` & `SigninUser`
@phughk phughk added the enhancement New feature or request label Mar 9, 2023
@phughk phughk added the stale label May 23, 2023
@phughk
Copy link
Contributor

phughk commented May 23, 2023

Hey @iDevelopThings ! Thanks for creating this - would it be possible to resolve the conflicts please? To make it easier to review. Thanks!

@timpratim
Copy link
Contributor

Hey @iDevelopThings gentle ping, can you fix the merge conflicts so that we can go ahead and review the PR?

@timpratim
Copy link
Contributor

Hey @iDevelopThings! Any updates on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants