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

Dexterity 2.0.x compatibility #154

Merged
merged 30 commits into from
Apr 5, 2013
Merged

Conversation

hvelarde
Copy link
Member

@hvelarde hvelarde commented Apr 2, 2013

No description provided.

ericof and others added 30 commits March 31, 2013 21:01
…ceDirectory as it is missing from five.grok 1.3.2
… with a merge at five.grok, http://svn.zope.org/five.grok/trunk/src/five/grok/meta.py?rev=123298&r1=112163&r2=123298

That merge removes the static resources grokker, maybe a not desired side effect of the merge. The static resource is documented by dexterity, and used by this package.
To fix this problem, if static resources grokker doesn't come back to five.grok, we can use a method also documented at dexterity, http://plone.org/products/dexterity/documentation/manual/developer-manual/advanced/static-resources
This way we use @@plone_portal_state/navigation_root_url as a base for our resources instead of the problematic static method.
hvelarde added a commit that referenced this pull request Apr 5, 2013
@hvelarde hvelarde merged commit 42922c4 into master Apr 5, 2013
@hvelarde
Copy link
Member Author

hvelarde commented Apr 5, 2013

closes #152 also

@hvelarde hvelarde deleted the dexterity-2.0.x-compatibility branch April 5, 2013 16:35
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 9fb143d on dexterity-2.0.x-compatibility into * on master*.

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

Successfully merging this pull request may close these issues.

4 participants