[aur-dev] [PATCH] Change File Request To Submit a Request

[Lukas Fleischer]

On Wed, 25 Nov 2015 at 03:26:03, Mark Weiman wrote:
> It looks like my last attempt to post this here failed, so I am trying
> again.  Ignore this if the first one appears.
> 
> Here is a patch that makes it Submit a Request.  I believe I also got
> the strings in the translation files.
> [...]

Great, thanks for the patch! It needs some polishing before it can be
applied, though...

* Please resubmit the patch inline (using git-send-email(1)) such that
  we can easily comment on specific parts. Commenting without any
  reference (like this) makes life harder for both the reviewer and the
  patch submitter.

* Change the commit message such that the subject line is <50 characters
  long (something like 'Reword "File Request"'?) and the details are
  mentioned in the message body. We usually write "Fixes FS#47167." when
  a commit fixes a bug, though it is not a must. Also, a sign-off is
  missing.

* Please drop all changes to the translation files. They are usually
  only updated before a new release is tagged.

* You also changed the title of the "File Request" page and the label of
  the submit button on the "File Request" page. I am not sure about the
  former but I think the latter is not a good idea. It should either
  keep saying "File Request" or be changed to "Submit Request". If we
  decide to change it, maybe using "Submit Request" everywhere makes
  sense because then we wouldn't need to translate two strings meaning
  the same thing.

Regards,
Lukas