← Back to team overview

mahara-contributors team mailing list archive

[Bug 1795096] [NEW] Turn delete button into button and add confirmation dialogue when deleting IdP

 

Public bug reported:

In bug #1716820 we introduced that the metadata of an IdP can be deleted
when not used anymore. That's been added to
/admin/extensions/pluginconfig.php?plugintype=auth&pluginname=saml

The confirmation dialogue is missing and the deletion is immediate.
Since we only have a "Save" button but no "Cancel", it can't be undone.
Since it is a distructive method, a confirmation pop-up would be useful
as we have for other delete actions.

The icon should also sit on a button. It doesn't need a heading as we
normally do not have headings for icons. If the heading is kept, the
button needs to move into the middle of the column to sit correctly
under the heading.

** Affects: mahara
     Importance: High
         Status: Confirmed

** Attachment added: "idps.png"
   https://bugs.launchpad.net/bugs/1795096/+attachment/5194273/+files/idps.png

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1795096

Title:
  Turn delete button into button and add confirmation dialogue when
  deleting IdP

Status in Mahara:
  Confirmed

Bug description:
  In bug #1716820 we introduced that the metadata of an IdP can be
  deleted when not used anymore. That's been added to
  /admin/extensions/pluginconfig.php?plugintype=auth&pluginname=saml

  The confirmation dialogue is missing and the deletion is immediate.
  Since we only have a "Save" button but no "Cancel", it can't be
  undone. Since it is a distructive method, a confirmation pop-up would
  be useful as we have for other delete actions.

  The icon should also sit on a button. It doesn't need a heading as we
  normally do not have headings for icons. If the heading is kept, the
  button needs to move into the middle of the column to sit correctly
  under the heading.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1795096/+subscriptions


Follow ups