Code review comment for lp:~adeuring/launchpad/bug-588276

Revision history for this message
Graham Binns (gmb) wrote :

> + "bug_reported_acknowledgement",
> ]
> custom_widget('bugtracker', ProductBugTrackerWidget)
>
> @@ -443,7 +446,7 @@
> else:
> private = False
>
> - notifications = ["Thank you for your bug report."]
> + notifications = [self.getAcknowledgementMessage(self.context)]
> params = CreateBugParams(
> title=title, comment=comment, owner=self.user,
> security_related=security_related, private=private,
> @@ -775,6 +778,47 @@
> })
> return guidelines
>
> + default_bug_reported_acknowledgement = "Thank you for your bug report."
> +
> + def getAcknowledgementMessage(self, context):
> + """An acknowlegement message displayed to the user."""
> + # If a given context has not a custom message, we go up in the

Minor grammatical nit pick: "does not have a" reads better than "has not a".

review: Approve (code)

« Back to merge proposal