https://drupal.org/sandbox/irinaz/2279929

git clone --branch master git.drupal.org:sandbox/irinaz/2279929.git views_exposed_filters_summary

Comments

moelius’s picture

Issue summary: View changes

There are some errors reported by automated review tools, you must check this http://pareview.sh/pareview/httpgitdrupalorgsandboxirinaz2279929git
FOUND 255 ERRORS AND 5 WARNINGS AFFECTING 107 LINES

rob.barnett’s picture

Issue summary: View changes

I would also suggest running your code through CodeSniffer as well. pareview.sh doesn't always catch what CodeSniffer does and you will need to cleanup whatever CodeSniffer finds in order to get accepted.
https://www.drupal.org/node/1419988. Best of luck.

irinaz’s picture

Issue summary: View changes

Thanks, I will work on this in August.

irinaz’s picture

http://pareview.sh/pareview/httpgitdrupalorgsandboxirinaz2279929git

-----------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
------------------------------------------------------

tim.plunkett’s picture

Opened #2337777: Coding standards fixes with some fixes.

PA robot’s picture

Status: Needs work » Closed (won't fix)

Closing due to lack of activity. If you are still working on this application, you should fix all known problems and then set the status to "Needs review". (See also the project application workflow).

I'm a robot and this is an automated message from Project Applications Scraper.

tim.plunkett’s picture

Status: Closed (won't fix) » Needs work

Bot accounts should be banned from d.o

PA robot’s picture

Status: Needs work » Closed (won't fix)

Closing due to lack of activity. If you are still working on this application, you should fix all known problems and then set the status to "Needs review". (See also the project application workflow).

I'm a robot and this is an automated message from Project Applications Scraper.

tim.plunkett’s picture

Status: Closed (won't fix) » Needs work

Booooo

klausi’s picture

Status: Needs work » Closed (won't fix)

@irinaz: does this need review? See the workflow: https://www.drupal.org/node/532400

Otherwise let's leave this closed until we hear back from the applicant.

tim.plunkett’s picture

Status: Closed (won't fix) » Postponed (maintainer needs more info)

Then let's use something that will show up in the queue (open issues will hide closed (won't fix).

irinaz’s picture

I will do more work on this module this upcoming week and post changes. sorry for delay. Tim, thanks!!

PA robot’s picture

Status: Postponed (maintainer needs more info) » Closed (duplicate)
Multiple Applications
It appears that there have been multiple project applications opened under your username:

Project 1: https://www.drupal.org/node/2476823

Project 2: https://www.drupal.org/node/2280911

As successful completion of the project application process results in the applicant being granted the 'Create Full Projects' permission, there is no need to take multiple applications through the process. Once the first application has been successfully approved, then the applicant can promote other projects without review. Because of this, posting multiple applications is not necessary, and results in additional workload for reviewers ... which in turn results in longer wait times for everyone in the queue. With this in mind, your secondary applications have been marked as 'closed(duplicate)', with only one application left open (chosen at random).

If you prefer that we proceed through this review process with a different application than the one which was left open, then feel free to close the 'open' application as a duplicate, and re-open one of the project applications which had been closed.

I'm a robot and this is an automated message from Project Applications Scraper.

avpaderno’s picture

Issue tags: -#views
Related issues: +#2476823: [D7] Views Exposed Filters Summary