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.

Sign up for GitHub

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

Jump to bottom

Users #48

Merged
merged 15 commits into from
Jun 6, 2016
Merged

Users #48

merged 15 commits into from
Jun 6, 2016

Conversation

Snugug
Copy link
Member

@Snugug Snugug commented Jun 2, 2016

User Authentication and Roles.

Changelog

**New**

* User Authentication
* Roles

Resolves: #44
Resolves #37

DCO 1.1 Signed-off-by: Sam Richard <sam@snug.ug>

@Snugug
Copy link
Member Author

Snugug commented Jun 2, 2016

Don't merge yet

@Snugug Snugug mentioned this pull request Jun 2, 2016
Closed
@Snugug
Copy link
Member Author

Snugug commented Jun 2, 2016

This resolves all of my comments from #44 except acl-knex which seems to be unmaintained and currently doesn't work with ACL 4.7+. Worth investigating, later

@Snugug Snugug added this to the Sprint 12: 5-25 milestone Jun 2, 2016
@Snugug
Copy link
Member Author

Snugug commented Jun 2, 2016

This is good to review. I'd like to be able to test both admin and content creator access on these URLs, but we can look at that later

@scottnath scottnath merged commit 425ad36 into punchcard-cms:master Jun 6, 2016
@Snugug Snugug mentioned this pull request Jun 6, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@scottnath scottnath

Labels
None yet
Projects
None yet
Milestone
Sprint 12: 5-25
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Snugug @scottnath