Dear Reviewer,

This module adds functionality to the Apache SOLR Drupal module, by allowing sites that share a SOLR index to filter the results to the site only. This is a feature that I didn't find in the standard apache solr module.

Project Page:
https://drupal.org/sandbox/rodolfovieira/2112599

Git Repo;
rodolfovieira@git.drupal.org:sandbox/rodolfovieira/2112599.git

LInk to reviews:
Not sure if applicable,
Here is a list of drupal issues posted:
https://drupal.org/project/issues/user

Github:
https://github.com/r-vieira/

Please let me know if you need further info.
Thanks
Rodolfo Vieira

Comments

genjohnson’s picture

Status: Needs review » Needs work

Hi :)

In the .info file, requires[] should be dependencies[] ( see https://drupal.org/node/542202).

Generally the function and file names match the module name. For example, apache_solr_multisite_filter.module instead of amps_apache_solr_site_filter.module.

There's an extra file in the repo that it looks like should be removed.

It looks like you're working on the master branch when you should be working on a version branch. See https://drupal.org/empty-git-master for steps to move from master.

alexverb’s picture

You should perform the next two tasks in it's order for people to easily clone your repository.

  1. You are still working on the master branch. You should switch to a major one. Just follow the instructions here:
    https://drupal.org/empty-git-master
  2. Your git repo message in the issue description should be a publicly clonable command. You can find this on your projects Version Control tab:
    https://drupal.org/project/2112599/git-instructions

After this people will start to look at your module...

PA robot’s picture

Issue summary: View changes
Status: Needs work » Closed (won't fix)

Closing 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.