Skip to content
This repository has been archived by the owner on Dec 12, 2021. It is now read-only.

Fix namespace split #668

Merged
merged 1 commit into from
Jul 2, 2012
Merged

Fix namespace split #668

merged 1 commit into from
Jul 2, 2012

Conversation

xinuc
Copy link
Contributor

@xinuc xinuc commented Jun 29, 2012

#ability.rb
can :index, "admin/dashboard"

this code will incorrectly try to load Dashboard class, not Admin::Dashboard class

@diegorv
Copy link

diegorv commented Jun 29, 2012

+1

@xinuc
Copy link
Contributor Author

xinuc commented Jul 2, 2012

@andhapp I've ported this pull request to master and submit pull request at #675

thanks

@andhapp
Copy link
Collaborator

andhapp commented Jul 2, 2012

@xinuc: Thanks a lot. I was going to do it at some point but I'm glad that you've already done it. Cheers.

ryanb added a commit that referenced this pull request Jul 2, 2012
@ryanb ryanb merged commit aed9f26 into ryanb:2.0 Jul 2, 2012
@ljulien
Copy link

ljulien commented Apr 11, 2013

Is there any chance of getting this released on the 1.6.x line? I could really use it.

@diegorv
Copy link

diegorv commented Apr 11, 2013

@ljulien, the last commit was 2 months ago... :/

Make a cancan fork and fix that for you...

It was my choice.. :/

ryanb added a commit that referenced this pull request May 3, 2013
@jaredbeck
Copy link

@diegorv, @ljulien, please note that this patch is now available in 1.6.10.

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

Successfully merging this pull request may close these issues.

6 participants