Closed (fixed)
Project:
Drupal.org CVS applications
Component:
co-maintainer application
Priority:
Normal
Category:
Task
Assigned:
Issue tags:
Reporter:
Created:
3 May 2010 at 19:08 UTC
Updated:
2 Nov 2018 at 15:53 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
pglatz commentedPlease use the updated versions of these from #5, thanks.
Comment #2
pglatz commentedComment #3
avpadernoHello, and thanks for applying for a CVS account. I am adding the review tags, and some volunteers will review your code, pointing out what needs to be changed.
As per http://drupal.org/cvs-application/requirements, the motivation message should be expanded to contain more details about the features of the proposed module, and it should include also a comparison with the existing solutions.
We then review a module per applicant; let us know which one you want to be reviewed.
Comment #4
pglatz commentedThanks. Please review the watchspy module.
Motivation: watchspy is a primary diagnostic tool for Drupal developers, and
contains very useful information. The user interface, however, is pretty bare
bones. It does allow filtering by type and severity, but not by date range. The
output is presented as a summary; to read the details of a message requires
opening a new browser tab or window. I find the clicking to read full message
texts can be a lot of work.
Watchspy eliminates much of the clicking by displaying full message details in
an expanded hidden div when the mouse is above the summary. You can stiull click
on the message to see the full details in another tab or window, but most of the
time the simple mouseover is all you need.
It has two additional features that can be useful:
1) Filter the date range of messages.
2) The ability to purge classes of message types from the log, handy if you
want to start a new test and look for specific messages.
It also has a settings page, where you can specify the number of rows per page
and control the appearance of the summary screen.
I am not aware of other modules that allow an expansion of watchdog message
details with a mouseover.
TODO:
1) Add filter by servity
Comment #5
pglatz commentedI just uploaded corrected zips of watchspy and node dump (duh, I had included .svn info with them). I also cleaned up some of the logic and styling for watchspy; please grab this one instead of the original version I posted.
Comment #6
pglatz commentedI have been asked to co-maintain Search Log and will need CVS access. Please see http://drupal.org/node/788746
Comment #7
avpadernoThank you for your contribution! I am going to update your account.
These are some recommended readings to help with excellent maintainership:
You can find more contributors chatting on the IRC #drupal-contribute channel. So, come hang out and stay involved.
Thank you, 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.
I thank all the dedicated reviewers as well.
Comment #10
avpaderno