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

able.to view helper: check access or assign access? #1

Open
gautamarora opened this issue May 3, 2012 · 4 comments
Open

able.to view helper: check access or assign access? #1

gautamarora opened this issue May 3, 2012 · 4 comments

Comments

@gautamarora
Copy link
Collaborator

Hi,
Is the use of the able.to helper to check if the user has access on a certain target or to assign access to a target.

From the docs, i expected it to check access, but it seems it assigns.

I have the fix in place if this is a bug, or else think another helper to check access will be useful.

@scottkf
Copy link
Owner

scottkf commented May 3, 2012

It should just check access, assigning should, use the abilities definition in the example. Though, as per this request and your others: I can not legally continue work on this project, though I can advise on it and will be happy to take pull requests and/or someone to takeover the project. It exists as it is today because I extracted what parts I could.

@gautamarora
Copy link
Collaborator Author

ok, i think there was a bug and instead of checking it was assigning the ability.
fixed at: gautamarora@6e8b09c

can issue a pull, if you think its right

@scottkf
Copy link
Owner

scottkf commented May 7, 2012

That's correct, go ahead and submit a pull request!

@sbabigian
Copy link

@gautamarora I hereby nominate you to take it over :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants