Details

    • Type: New Feature
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Development
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Epic Link:
    • Application Requirement:
      KFS
    • Sprint:
      Middleware 2.5.2 Sprint 3
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required
    • Code Review Status:
      Not Required
    • Include in Release Notes?:
      Yes
    • Story Points:
      5

      Description

      KFS has a requirement to be able to email an attachment, however it doesn't appear the Rice code we use to send emails (either org.kuali.rice.core.api.mail.Mailer or org.kuali.rice.krad.service.MailService) supports attachments. We would like this ability added to Rice so we don't have to roll our own email service in KFS to support attachments.

        Attachments

          Issue Links

            Activity

            Hide
            bhutchinson Bryan Hutchinson added a comment -

            This changeset contains the code I wrote for KFS to workaround the lack of this feature:
            https://fisheye.kuali.org/changelog/kfs?cs=44570

            The main things to look at are the AttachmentMail* artifacts under the org.kuali.kfs.sys package tree. I extended the Rice classes, but the code could certainly be merged into the Rice classes to add this feature. Is that something the KFS team could work with the Rice team on contributing?

            Thanks,
            Bryan

            Show
            bhutchinson Bryan Hutchinson added a comment - This changeset contains the code I wrote for KFS to workaround the lack of this feature: https://fisheye.kuali.org/changelog/kfs?cs=44570 The main things to look at are the AttachmentMail* artifacts under the org.kuali.kfs.sys package tree. I extended the Rice classes, but the code could certainly be merged into the Rice classes to add this feature. Is that something the KFS team could work with the Rice team on contributing? Thanks, Bryan
            Hide
            jksmith James Smith added a comment -

            As per a discussion with Claus and Jonathan this morning, we'd like to also add the requirements of KFSCNTRB-1796 to this - which is to say, in non-production environments, set the from address on the e-mail to the Non-Production notification list (see the last comment of the jira).

            Show
            jksmith James Smith added a comment - As per a discussion with Claus and Jonathan this morning, we'd like to also add the requirements of KFSCNTRB-1796 to this - which is to say, in non-production environments, set the from address on the e-mail to the Non-Production notification list (see the last comment of the jira).
            Hide
            cniesen Claus Niesen added a comment -

            @James, I created KULRICE-14095 for the non-production from address modification

            Show
            cniesen Claus Niesen added a comment - @James, I created KULRICE-14095 for the non-production from address modification
            Hide
            jksmith James Smith added a comment -

            Great - thank you!

            Show
            jksmith James Smith added a comment - Great - thank you!
            Hide
            shahess Shannon Hess added a comment -

            I created https://fisheye.kuali.org/cru/rice-689 to reivew my changes. The most notable change I made from Bryan's code is that I allowed for multiple attachments.

            Show
            shahess Shannon Hess added a comment - I created https://fisheye.kuali.org/cru/rice-689 to reivew my changes. The most notable change I made from Bryan's code is that I allowed for multiple attachments.
            Hide
            cniesen Claus Niesen added a comment -

            This change was rolled back. See KULRICE-14252 for instructions to reapply.

            Show
            cniesen Claus Niesen added a comment - This change was rolled back. See KULRICE-14252 for instructions to reapply.

              People

              • Assignee:
                shahess Shannon Hess
                Reporter:
                bhutchinson Bryan Hutchinson
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: