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

Installation #54

Open
petermacdonald opened this issue Nov 3, 2018 · 3 comments
Open

Installation #54

petermacdonald opened this issue Nov 3, 2018 · 3 comments

Comments

@petermacdonald
Copy link

Using Islandora: 7x-1.11

I updated "islandora_context" module today. I removed the old version and cloned the current one. I enabled it along with the "context" module.

I cleared all caches several times, but I don't see "Context" listed in "Structure" menu.

I enabled it on another site as well (we use Drupal's multi-site) with the same problem.

Also, both sites have the following "notice" from "ctools" in the watchdog log which is related to this module, but might not have any bearing on the menu problem

Plugin islandora_context_condition_no_book_viewer of plugin type context:plugins points to nonexistent file sites/all/modules/islandora_context/plugins/islandora_context_condition_no_book_viewer.inc for class handler handler.

Thanks for your help.
Peter

@mjordan
Copy link
Collaborator

mjordan commented Nov 3, 2018

@petermacdonald did you also enable the Context UI module? That is the one that provides the menu link. If enabling that module fixes your problem, I'll add it to the README.

About the warning, I'll fix that, it's caused by some code that should have been removed a while ago.

@petermacdonald
Copy link
Author

petermacdonald commented Nov 4, 2018 via email

@mjordan
Copy link
Collaborator

mjordan commented Nov 4, 2018

@petermacdonald I've updated the README that address the initial focus of this issue. Thanks for pointing out that the README needed mention of Context UI. (I've also removed the crufty code that was producing that warning.)

I added the ability to hide the "In collections div" just a couple of weeks ago, and there's a good chance you are the first person to test it other than me. Would you mind moving discussion over to #53 (which I will repoen) so I can close this issue and mark it resolved? Feel free to just copy and paste your last comment.

I appreciate the feedback. I don't take it as negative, I take it as constructive. Also, I'm about to head off to the iCamp in San Diego on Tuesday morning so might not get back to you promptly while I'm there, but I'll try.

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

2 participants