patch for README

CommentFileSizeAuthor
google-connector.patch1.67 KBamitaibu

Comments

frans’s picture

frans commented
Assigned: Unassigned » frans
Status: Needs review » Active

Tnx!

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
commented

Why set back to active? :)

frans’s picture

frans commented

Ow, when I assign, I put it on active... just to remember myself that it needs work (yeah patching is work too you know...)

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

If there's a patch that needs review then the status should be "needs review." If you have a different system for statuses than the standard you're likely to confuse a lot of people. I won't change the status back to "needs review" but encourage you to consider doing that.

I think everyone respects that having a patch committed involves a lot of work by the module maintainer and I am thankful for your work.