Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-30000

Site creation notification emails should use the site title instead of the site ID

    XMLWordPrintable

    Details

    • Type: (Deprecated) Contributed Patch
    • Status: RESOLVED
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 11.0
    • Fix Version/s: 11.0
    • Component/s: Site Info
    • Labels:
      None
    • Conversion Script Required:
      Yes

      Description

      There are too many instances of the site ID in the two site creation notification emails (sitemanage.notifySiteCreation, sitemanage.notifySiteCreation.confirmation). For the majority of users, they don't care or even know what the site ID is. The relevant information is the site title.

      The linked PR addresses this issue, as well as some grammatical errors in the templates. I've left a single occurrence of the site ID in the message body for troubleshooting purposes, etc. I've switched out the domain in favour of the ui.service for better readability, but will fall back to the domain if ui.service is not set in sakai.properties. Also, cleaned up ETSUserNotificationProviderImpl.java a bit while I was in there.

      Existing instances will need to manually update the template via SQL. The update SQL is contained in /reference/docs/conversion/sakai_11_mysql_conversion.sql or /reference/docs/converstion/sakai_11_oracle_conversion.sql. Any new instance will get the new template automatically.

        Gliffy Diagrams

          Zeplin

            Attachments

              Issue Links

                Activity

                  People

                  Assignee:
                  bjones86 Brian Jones
                  Reporter:
                  bjones86 Brian Jones
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                    Dates

                    Created:
                    Updated:
                    Resolved:

                      Git Integration