#5789 closed defect (fixed)
[patch] Change description on notification admin panel
| Reported by: | Sebastian Krysmanski | Owned by: | Steffen Hoffmann |
|---|---|---|---|
| Priority: | normal | Component: | AccountManagerPlugin |
| Severity: | major | Keywords: | web_ui trivial |
| Cc: | John Hampton | Trac Release: | 0.11 |
Description
On the "Notification" panel it reads: "These are the email addresses that get notified of the above actions." This is, however, not entirely correct as usernames are allowed as well. Furthermore it should be stated that recipients are separated by space (and not be comma or anything else).
I've attached a patch that corrects this description.
Attachments (1)
Change History (6)
Changed 15 years ago by
| Attachment: | notification-panel-desc.patch added |
|---|
comment:1 Changed 14 years ago by
| Cc: | John Hampton added; anonymous removed |
|---|---|
| Keywords: | web_ui trivial added |
| Priority: | low → normal |
| Summary: | Change description on notification panel → [patch] Change description on notification panel |
comment:2 Changed 14 years ago by
| Owner: | changed from Matt Good to Steffen Hoffmann |
|---|
Patch is ready. But I'll break this out of the i18n topic mentioned above for patch code clarity.
comment:3 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
(In [9273]) AccountManagerPlugin: Improve description in notification admin form, closes #5789.
comment:4 Changed 14 years ago by
(In [9318]) AccountManagerPlugin: Add basic i18n setup and message markup, refs #5789.
While doing message markup some msgid changes where incorporated as well. Initial message extraction has been done and German translations added. There is still more work required, i.e. to send localized email messages.
comment:5 Changed 14 years ago by
| Summary: | [patch] Change description on notification panel → [patch] Change description on notification admin panel |
|---|
Thanks for the patch. This should be addressed really soon. Best done before/together with adding i18n/l10n to this plugin (see #6803).