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

change context to accounts and add roles table #101

Closed
wants to merge 8 commits into from

Conversation

GettyOrawo
Copy link
Contributor

@GettyOrawo GettyOrawo commented Mar 25, 2019

Done

Change context from User to Accounts to avoid redundancy since there's already a module called User.
Create a form with functionality for creating a role.

  • Roles should not be blank
  • Roles should be unique
    Context tests for the above functionality

To be done

  • Every created user should belong to a role.
  • A role can have many users.

roles

@sigu sigu added the review label Mar 25, 2019
Copy link
Member

@sigu sigu left a comment

Choose a reason for hiding this comment

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

Great one, opening an issue to resolve my comment. Should not be done on this PR

lib/nailinda/accounts/accounts.ex Show resolved Hide resolved
@GettyOrawo
Copy link
Contributor Author

This merge request has been opened in another branch. closing... #109

@GettyOrawo GettyOrawo closed this Mar 26, 2019
@sigu sigu removed the review label Mar 26, 2019
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

2 participants