Uploaded image for project: 'Continuum'
  1. Continuum
  2. CONTINUUM-2107

Send on SCM Failure is duplicated on the Add/Edit Mail Notifier page

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.3.1 (Alpha)
    • Fix Version/s: 1.3.2 (Beta)
    • Component/s: Notifier - Mail
    • Labels:
      None

      Description

      The text 'Send on SCM Failure' appears twice on the Add/Edit mail notifier page.

        Issue Links

          Activity

          Hide
          wsmoak Wendy Smoak added a comment -

          Applied patch from CONTINUUM-2113 in r750968. Thanks, Nap!

          Show
          wsmoak Wendy Smoak added a comment - Applied patch from CONTINUUM-2113 in r750968. Thanks, Nap!
          Hide
          wsmoak Wendy Smoak added a comment -

          See patch on duplicate issue CONTINUUM-2113

          Is removing the second checkbox the right thing to do, or was it intended to be a checkbox for something else?

          Removing it will fix this issue, but I wonder if something is missing.

          Show
          wsmoak Wendy Smoak added a comment - See patch on duplicate issue CONTINUUM-2113 Is removing the second checkbox the right thing to do, or was it intended to be a checkbox for something else? Removing it will fix this issue, but I wonder if something is missing.
          Hide
          jomm Jose Morales Martinez added a comment -

          This happens in all notifiers except Wagon Notifier.

          In adittion, I think the label should be modified. Replace 'Send On SCM Failure' for 'Send on SCM Failure'. This way all labels follow the same pattern (lowercase).

          Show
          jomm Jose Morales Martinez added a comment - This happens in all notifiers except Wagon Notifier. In adittion, I think the label should be modified. Replace 'Send On SCM Failure' for 'Send on SCM Failure'. This way all labels follow the same pattern (lowercase).

            People

            • Assignee:
              wsmoak Wendy Smoak
              Reporter:
              wsmoak Wendy Smoak
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development