Project d.o page: https://drupal.org/sandbox/clrh/1866840
Website: http://wheke.org
Branch: 7.x-2.x on http://git.drupal.org/sandbox/clrh/1866840.git
What is it : A drupal installation profile and distribution for libraries
Where is it and how I build it:
$ git clone --branch 7.x-2.x http://git.drupal.org/sandbox/clrh/1866840.git wheke
$ drush make wheke/build-wheke.make $DIR_INSTALL --working-copy
$ cd $DIR_INSTALL
$ drush si -y wheke --account-name=$ACCOUNT_NAME --account-pass=$ACCOUNT_PASS --db-url='mysql://'$MYSQL_USER':'$MYSQL_PASS'@'$MYSQL_SERVER'/'$MYSQL_DB --locale=fr
First, we provided our first module called opac. The opac module enables libraries to integrate their catalog into Drupal allowing importing records, make advanced searches with faceted results, circulation task etc ... This module is intended to work with any ILS by using connectors (koha_connector is the first to be alive). Everyone can make its own by creating a php file. We are working too with pazpar2 for federated search with mkdru and plan to develop an ERM soon.
We needed a Drupal distribution with modules preconfigured for helping libraries to build their website. It is the aim of this project. We are building a website dedicated: http://wheke.org
Comment | File | Size | Author |
---|---|---|---|
#11 | simplesearch_available.png | 39.67 KB | Claire Hernandez |
#11 | revue_ct-journal-.png | 187.46 KB | Claire Hernandez |
#11 | opac_user_settings.png | 108.98 KB | Claire Hernandez |
#11 | opac_server_features.png | 18.97 KB | Claire Hernandez |
#11 | opac_items_customisation.png | 66.7 KB | Claire Hernandez |
Comments
Comment #1
vijaycs85Thanks for your effort to provide a distribution/platform to libraries. Here are my few initial findings:
1. http://pareview.sh/pareview/httpgitdrupalorgsandboxclrh1866840git - There are some basic coding standard complaints that we need to fix. once you fix the code, you can repeat the test to get the new code base status.
2. Other than downloading the real code, I couldn't find whether it is module/theme/profile. Can we make it clear in description please?
Comment #2
Claire Hernandez CreditAttribution: Claire Hernandez commented1. The branch where I worked is 7.x-2.x it should pass ventral, dcs and verify-makefile: http://pareview.sh/pareview/httpgitdrupalorgsandboxclrh1866840git-7x-2x
2. I changed the issue description to make it clearer
Thanks.
Comment #3
vijaycs85Thank for your update @Claire Hernandez. I will install and check the functionality later today or tomorrow.
Comment #4
PA robot CreditAttribution: PA robot commentedWe are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)
Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #5
vijaycs85Manually installed (As specified in 'Where is it and how I build it:') and here is my findings:
1. Got few warning message on install
2. After logged in, can see list of modules enabled by default on modules list page.
Comment #6
vijaycs85duplicate of #5
Comment #6.0
vijaycs85description changed
Comment #7
Claire Hernandez CreditAttribution: Claire Hernandez commentedWarnings should have disapear because I fixed the guilty module (flag_cart), released it and updated the branch with the new 1.1 flag_cart version.
Thanks.
Comment #8
vijaycs85Ok, lets update the tag to someone manually instal this and produce result.
Comment #9
Bès CreditAttribution: Bès commentedHi here,
just go throw the installation process. First remark, it's based on Drupal 7.22, may need to update to 7.23.
No error, except at the end but it's due to my computer configuration.
Will now try to play with the distribution
Comment #10
Claire Hernandez CreditAttribution: Claire Hernandez commentedI fixed things:
* core version
* security version of ds
Regards.
Comment #10.0
Claire Hernandez CreditAttribution: Claire Hernandez commentedfix a mini mistake in "how to"
Comment #11
Claire Hernandez CreditAttribution: Claire Hernandez commentedComment #12
Claire Hernandez CreditAttribution: Claire Hernandez commentedTo help "functionnal test" (not easy I know, because you should setup a library integrated system to use it completly), I will try to give you some hints:
One built you can see:
- a basic but theme wich match with wheke-demo ad wheke.org website (should be responsive and based on prt)
- custom configurations for a unimarc lis (you can harvest a lis like koha to have bibliographic records as node in drupal)
- some specific modules as search
- you can see opac in configuration
- custom configuration for authentication between drupal and lis and borrower (member of library) recognition
- biblio and journal ct with associated displays
- item customisations
- authorities configurations
I think I am not exaustive, but you can look forward I hope :)
Hope it helps !
Comment #13
kscheirerOtherwise looks good to me. I did not actually install the distro though.
----
Top Shelf Modules - Crafted, Curated, Contributed.
Comment #14
Claire Hernandez CreditAttribution: Claire Hernandez commentedHello,
Thanks for feedbacks.
Comment #15
kscheirerHi Claire,
I'm happy to promote this distribution to a full release.
However, it doesn't seem like you've written any of the code, so we cannot grant you "git vetted user" status to promote your own projects.
Please let us know how you'd like to proceed and if I've made an error.
Comment #16
Claire Hernandez CreditAttribution: Claire Hernandez commentedHi Karl,
Thanks for review.
This repository has not a lot of "code", it is a lot of packaging, modules versionning, not just php instructions. However, I am the main maintainer of wheke distribution (131 commits https://drupal.org/sandbox/clrh/1866840). I am involved into main opac module (and others) and wheke team as code reviewer, documentation writer, tester, translator, infrastructure maintainer, architecture validator, I do releases too and fix things when they need it. I know perhaps I am not a "developper" as you mean, but I think my implication is not insignificant.
It is up to you to decide ;)
Comment #17
Claire Hernandez CreditAttribution: Claire Hernandez commentedI think status should changed (not sure, I hope I don't do a mistake)
Comment #18
Claire Hernandez CreditAttribution: Claire Hernandez commentedWhat does it miss to this issue ? 2 months ago, you were ready to promote this project. Did you change your mind ? What could I do to help ?
Comment #19
jneubert CreditAttribution: jneubert commentedHi Claire,
Your distribution sounds quite interesting, so I tried to install it, following your description above. All went fine untill the step
The database itself and a number of tables have been created, so the basic setup seems to work.
Two additional comments:
1) During make, Drupal 7.23 is downloaded. If this isn't updated later on automatically, you perhaps should consider downloading always the current version (in the meantime, Drupal 7.24 and 7.26 fixed security issues).
2) From the build description above it was not clear to me if mysql as a database and language=fr are restrictions, or just example values. If English is supported, or other databases, it could be helpful to mention that explicitely.
Cheers, Joachim
Comment #20
jneubert CreditAttribution: jneubert commentedComment #21
PA robot CreditAttribution: PA robot commentedClosing due to lack of activity. Feel free to reopen if you are still working on this application (see also the project application workflow).
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #22
Claire Hernandez CreditAttribution: Claire Hernandez commentedI fixed a mistake in a linked module (simplesearch).
So back to the last status "reviewed & tested by the community".
I should me more present than these last 4 months...
Regards.
Comment #23
gisleThe automatic review found one error affecting one line. Please see: PAreview for details. This is trivial to fix, but need to be fixed before a release.
Comment #24
Claire Hernandez CreditAttribution: Claire Hernandez commented>> b85dcea README.txt file change regarding the template https://drupal.org/node/2181737
>> Now default branch is 2.x
Pareview is now clean !
Regards.
Comment #25
gislePlease note: You're not supposed to change the status to from "Needs work" to Reviewed & tested by the community" (RTBC) yourself (from the look at #17, #22, #24, it looks as if you're confused about this).
After you think you've fixed all the problems pointed out in a review, you change the status from "Needs work" to "Needs review". This indicates that you think the project is ready for a new review. You're not supposed to review your own project.
After the reviewer has had time to review the new version, he or she has two main options: To point out that there are still points that need to fixed, and set the status to "Needs work". Or to set the status to "Reviewed & tested by the community" to indicate that the reviewer now thinks the project is ready to be promoted into a full project. Note that this status in no guarantee of promotion. There may be other reviewers that disagree, and that may push it back down to "Needs work".
Comment #26
Claire Hernandez CreditAttribution: Claire Hernandez commentedOups... Ok I understand, thanks Gisle.
Comment #27
gisleThere are IMHO no more blockers. Moving to RTBC. Note that promotion will not happen until a git administrator has given this a second set of eyes.
Also note that it it does not follow the guidelines for project length and complexity (see also #15). It usually entails that project is promoted to full project status, but the applicant does not gain git vetted stattus (but the git administrator makes the final call on this as well).
Comment #28
gisleAdded tag.
Comment #29
kscheirerSorry for my absence Claire! I think your Drupal history shows enough work, and you're even listed as a mentor by another contributor with many commits. Checked for security, licensing, and account issues, I don't see any.
Thanks for your contribution, Claire Hernandez!
I updated your account so you can promote this to a full project and also create new projects as either a sandbox or a "full" project.
Here are some recommended readings to help with excellent maintainership:
You can find lots more contributors chatting on IRC in #drupal-contribute. So, come hang out and stay involved!
Thanks, also, for your patience with the review process. Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.
Thanks to the dedicated reviewer(s) as well.