[OpenNMS/opennms] ec2657: NMS-20118: Event Notifications tab

  Branch: refs/heads/jira/NMS-20118-event-notifications
  Home:   https://github.com/OpenNMS/opennms
  Commit: ec2657abc67c5a8a3575115fca1ce62da98e5b13
      https://github.com/OpenNMS/opennms/commit/ec2657abc67c5a8a3575115fca1ce62da98e5b13
  Author: Jose Anes <[email protected]>
  Date:   2026-08-03 (Mon, 03 Aug 2026)

  Changed paths:
    M opennms-webapp-rest/src/main/java/org/opennms/web/rest/v1/NotificationConfigRestService.java
    M opennms-webapp-rest/src/test/java/org/opennms/web/rest/v1/NotificationConfigRestServiceIT.java
    M ui/src/components/AdminNotifications/ConfigureNotificationsDialog.vue
    A ui/src/components/AdminNotifications/EventNotificationEditorDialog.vue
    A ui/src/components/AdminNotifications/EventNotificationsTable.vue
    M ui/src/services/index.ts
    M ui/src/services/notificationConfigService.ts
    M ui/src/stores/notificationConfigStore.ts
    M ui/src/types/notificationConfig.ts
    M ui/tests/stores/notificationConfigStore.test.ts

  Log Message:
  -----------
  NMS-20118: Event Notifications tab

Replaces the event-notifications placeholder with the tab content: the
notification list with per-notification on/off toggles, an add/edit
dialog with a searchable UEI picker fed by the event configuration, and
delete with confirmation. The /rest/notification-config
event-notifications endpoints wrap the notifications.xml factory the
legacy wizard uses.

Deleting the last event notification is rejected (notifications.xsd
requires at least one, and the schema-validated save would diverge
memory from the file until a restart), and validation covers every field
whose asserting setter could kill the factory's in-place update chain,
including status and parameter names/values.


  Commit: a21faee2d06bc1b5ac029848dd3465040996e9a2
      https://github.com/OpenNMS/opennms/commit/a21faee2d06bc1b5ac029848dd3465040996e9a2
  Author: Jose Anes <[email protected]>
  Date:   2026-08-03 (Mon, 03 Aug 2026)

  Changed paths:
    M ui/src/components/AdminNotifications/EventNotificationsTable.vue
    M ui/src/services/notificationConfigService.ts

  Log Message:
  -----------
  NMS-20118: surface the delete guard and disable deleting the last notification

Deleting an event notification now surfaces the server's reason instead of a
generic failure, so the "last event notification cannot be deleted" guard reads
as an explanation rather than an unexplained error. The Delete button is disabled
when only one notification remains, with a title pointing the user at turning it
off instead.


  Commit: 6e5951fdb7b3ad07fa7f3bbfd4af1655542f2db2
      https://github.com/OpenNMS/opennms/commit/6e5951fdb7b3ad07fa7f3bbfd4af1655542f2db2
  Author: Jose Anes <[email protected]>
  Date:   2026-08-03 (Mon, 03 Aug 2026)

  Changed paths:
    M ui/tests/stores/notificationConfigStore.test.ts

  Log Message:
  -----------
  NMS-20118: event-notifications loader reports real success

The event-notifications tab loader now returns every([...]).every(Boolean)
instead of a literal true: getEventNotifications returns null on failure and the
store reports a boolean, so a failed fetch retries instead of latching the default
tab on "No event notifications configured." Rebased onto the updated base. Adds a
store test pinning the failure contract.


Compare: https://github.com/OpenNMS/opennms/compare/e62bc00c0ade...6e5951fdb7b3

To unsubscribe from these emails, change your notification settings at https://github.com/OpenNMS/opennms/settings/notifications


_______________________________________________
Please read the OpenNMS Mailing List FAQ:
http://www.opennms.org/wiki/index.php?page=MailingListFaq
opennms-cvs mailing list

To *unsubscribe* or change your subscription options, see the bottom of this page:
https://lists.sourceforge.net/lists/listinfo/opennms-cvs