User roles and cleanup #109
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes issue number
fixes #110
Describe the changes made on this PR
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.
Context tests for the above functionality
To be done
Visuals
GIF
Reviews
I have formated by code as expected
We have a handy tool that helps you check if your elixir code is well formatted. If the tool passes on your local build
then it will definitly pass our linting checks online
I have filled in all the required information above
I have marked my work as
ready for review