Project

General

Profile

Refactor #18681

notification subject should be part of the notification model, not notification blueprint

Added by Ohad Levy over 4 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Notifications
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

When creating a notification, it is expected that each notification may contain a specific information about an object, for example a host.
in that case, its important to be able to link to the host specific action which is define by both the host ID and type.


Related issues

Related to Foreman - Feature #17545: Add required rails models and JSON endpoints to UI notificationClosed2016-12-01

Associated revisions

Revision a36689ab (diff)
Added by Ohad Levy over 4 years ago

fixes #18681 - moves polymorphic subject into notification object

also adds host build, destroyed and missing owner UI notifications

History

#1 Updated by The Foreman Bot over 4 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4339 added

#2 Updated by Ohad Levy over 4 years ago

  • Related to Feature #17545: Add required rails models and JSON endpoints to UI notification added

#3 Updated by Ohad Levy over 4 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#4 Updated by Ohad Levy over 4 years ago

  • Legacy Backlogs Release (now unused) set to 209

Also available in: Atom PDF