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

Padrino::Admin::AccessControl detect the model name of accounts from settings #729

Closed
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@aereal
Contributor

aereal commented Dec 4, 2011

Padrino::Admin::AccessControl uses `Account' model for storing accounts.

It's difficult to change (e.g. Account -> User) it because it's hard coded.

@DAddYE

This comment has been minimized.

Show comment
Hide comment
@DAddYE

DAddYE Dec 5, 2011

Member

Nice! What is the status of this pull-request?

Member

DAddYE commented Dec 5, 2011

Nice! What is the status of this pull-request?

@aereal

This comment has been minimized.

Show comment
Hide comment
@aereal

aereal Dec 5, 2011

Contributor

You says that is this pull-request bug-fix or feature-request?

If so, this is feature-request.

Contributor

aereal commented Dec 5, 2011

You says that is this pull-request bug-fix or feature-request?

If so, this is feature-request.

@DAddYE

This comment has been minimized.

Show comment
Hide comment
@DAddYE

DAddYE Dec 20, 2011

Member

sorry @aereal I asked this because aereal@f57f19d revert your commit... why?

Member

DAddYE commented Dec 20, 2011

sorry @aereal I asked this because aereal@f57f19d revert your commit... why?

@DAddYE DAddYE closed this in 92d20d7 Jan 22, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment