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

Contact Us tool should use mail.support property if no specific "contact points" are set

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: OPEN
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 11.0
    • Fix Version/s: None
    • Component/s: Feedback Tool
    • Labels:
    • 11 status:
      Please Merge
    • Test Plan:
      Hide

      Test A

      1/ feedback.technicalAddress should not be set in the properties file & mail.support should be set to (say) fred.bloggs@geemayle.com

      2/ Launch the Contact Us tool, click on the link in the second or third box (doesn’t matter), complete the fields and press send. There should NOT be a message about the email address not being set in the second and third boxes.

      3/ look in the logs and see that the message was sent

      Test B

      1/ feedback.technicalAddress should set to (say) john.doe@geemayle.com in the properties file & mail.support should be set to (say) fred.bloggs@geemayle.com

      2/ Launch the Contact Us tool, click on the link in the second or third box (doesn’t matter), complete the fields and press send. There should NOT be a message about the email address not being set in the second and third boxes.

      3/ look in the logs and see that the message was sent to John Doe and not Fred Bloggs

      Test C

      1/ feedback.technicalAddress & & mail.support should not be set

      2/ Launch the Contact Us tool, check there is a message about the email address not being set in the second and third boxes.

      Show
      Test A 1/ feedback.technicalAddress should not be set in the properties file & mail.support should be set to (say) fred.bloggs@geemayle.com 2/ Launch the Contact Us tool, click on the link in the second or third box (doesn’t matter), complete the fields and press send. There should NOT be a message about the email address not being set in the second and third boxes. 3/ look in the logs and see that the message was sent Test B 1/ feedback.technicalAddress should set to (say) john.doe@geemayle.com in the properties file & mail.support should be set to (say) fred.bloggs@geemayle.com 2/ Launch the Contact Us tool, click on the link in the second or third box (doesn’t matter), complete the fields and press send. There should NOT be a message about the email address not being set in the second and third boxes. 3/ look in the logs and see that the message was sent to John Doe and not Fred Bloggs Test C 1/ feedback.technicalAddress & & mail.support should not be set 2/ Launch the Contact Us tool, check there is a message about the email address not being set in the second and third boxes.

      Description

      At the moment the contact us (aka Feedback) tool will display "sub-optimally" if any of the middle 2 'contact points' are unset in the feedback tool properties file, ie the contact email for boxes 2&3. What should happen is the "mail.support" property should be used by default but it can be over-ridden by specific local (Feedback tool) property

        Gliffy Diagrams

          Zeplin

            Attachments

            1. Help_screenshot_qa.png
              251 kB
              Neal Caidin
            2. No properties.png
              64 kB
              Nick Wilson

              Issue Links

                Activity

                  People

                  Assignee:
                  nickwilson Nick Wilson
                  Reporter:
                  adamm Adam Marshall
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  7 Start watching this issue

                    Dates

                    Created:
                    Updated:

                      Git Integration