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

Create organization #68

Merged
merged 13 commits into from
Nov 27, 2023
Merged

Create organization #68

merged 13 commits into from
Nov 27, 2023

Conversation

algibson1
Copy link
Contributor

@algibson1 algibson1 commented Oct 23, 2023

Explain changes:

Adds functionality to register/create an organization in the BE database
Note: Does NOT yet include functionality for passwords and secure login

What does this PR do?

  • Feature
  • Bug fix
  • Refactor
  • Styling
  • Other:

Necessary checkmarks:

  • All Tests are Passing
  • The code will run locally

Testing Changes

  • No Tests have been changed/added
  • Some Tests have been changed/added
  • All of the Tests have been changed(Please explain)

Issues this closes

#57

@algibson1 algibson1 marked this pull request as ready for review October 31, 2023 15:53
argument :share_email, Boolean, required: false


def resolve(**args)
Copy link
Contributor

Choose a reason for hiding this comment

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

Love how DRY **args makes this 🥳

Copy link
Contributor

@Dav1s-Ops Dav1s-Ops left a comment

Choose a reason for hiding this comment

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

Register mutation and tests pass and look great! Nice work

@Dav1s-Ops Dav1s-Ops merged commit f23b538 into main Nov 27, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants