Hi, I am working on this at this very moment :) I would like to submit my proposed guidelines to the doc list for discussion prior to submitting as a patch, so please go ahead and apply Stefan's patch now if that makes sense. Djun On 12 May 2005, at 11:32 PM, stefan nagtegaal wrote:
Issue status update for http://drupal.org/node/22652
Project: Drupal Version: cvs Component: base system Category: bug reports Priority: normal Assigned to: Anonymous Reported by: stefan nagtegaal Updated by: stefan nagtegaal Status: patch
I did not see a patch of Djun yet.. Maybe it's better to let him build hi patch on top of this one?
Some documentation and guidelines are better than none at all...
stefan nagtegaal
Previous comments: ----------------------------------------------------------------------- -
May 12, 2005 - 10:32 : stefan nagtegaal
I updated the Doxygen comments of drupal_set_message() so it reflects the latest changes and we can keep the messages consistent..
As noted in the Doxygen comments, we *always* begin a message with a Capital letter, and end the message with a period ('.').
I hope one of you guys can spellcheck the documentation for me, because... Well, you probably know English isn't my strongest point.. ;-)
Please review, update/comment and apply...
----------------------------------------------------------------------- -
May 12, 2005 - 10:32 : stefan nagtegaal
Attachment: http://drupal.org/files/issues/drupal_set_message.patch (1.26 KB)
And, unfortunatly, I forgot the patch... :-(
----------------------------------------------------------------------- -
May 12, 2005 - 16:42 : Dries
I believe Djun had some additional guidelines.