Continuum

Missing confirmation when deleting a project-level notifier from the group-level Notifiers tab

Details

  • Complexity:
    Intermediate
  • Number of attachments :
    1

Description

On the Notifiers tab, the bottom section lists notifiers that were added for individual projects.

These can be deleted by clicking the red X out to the right, but there is no chance to confirm prior to the delete.

  1. Text File
    CONTINUUM-2135.patch
    19/Mar/09 6:20 PM
    3 kB
    Jose Morales Martinez

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Jose Morales Martinez added a comment -

Currently, the behavior is as follows:

  • If you delete a group level Notifier, there is a question for confirmation.
  • If you delete a project level Notifier,it doesn't ask for confirmation.

What would be the behavior? You always have to require confirmation?

Show
Jose Morales Martinez added a comment - Currently, the behavior is as follows: If you delete a group level Notifier, there is a question for confirmation. If you delete a project level Notifier,it doesn't ask for confirmation. What would be the behavior? You always have to require confirmation?
Hide
Permalink
Wendy Smoak added a comment -

Yes, there should always be a chance to confirm before something is deleted.

I changed the subject to "Missing confirmation...". Does that make more sense?

Show
Wendy Smoak added a comment - Yes, there should always be a chance to confirm before something is deleted. I changed the subject to "Missing confirmation...". Does that make more sense?
Hide
Permalink
Jose Morales Martinez added a comment -

I hope this patch solve the bug.

Show
Jose Morales Martinez added a comment - I hope this patch solve the bug.
Hide
Permalink
Maria Catherine Tan added a comment -

Fixed in revision 756532. Thanks!

Show
Maria Catherine Tan added a comment - Fixed in revision 756532. Thanks!

People

  • Assignee:
    Maria Catherine Tan
    Reporter:
    Wendy Smoak
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: