Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Notification PaperTrail #1

Open
Luckeu opened this issue Jan 3, 2022 · 0 comments
Open

Notification PaperTrail #1

Luckeu opened this issue Jan 3, 2022 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@Luckeu
Copy link
Contributor

Luckeu commented Jan 3, 2022

Many systems require a persistent papertrail of events and actions for traceability.

Currently TeamsConnector does not log or store a record when a notification was sent and whether this was successful or not.

To fix this a logger should be configurable which allows the output to different targets.

  • stdout
  • File
  • Rails ActiveRecord
  • Other (user configurable)

Concept

During configuration a logger can be defined. The interface is similar to a Rails ActiveRecord to allow easy integration without a layer in between.

This logger is a class that defines the class method ::create for initializing a new notification entry. For persistent papertrails the result must contain an id that will be passed to the template (e.g to display it in the notification).
The entry will be updated after the HTTP request is made. For this the class must support ::find(id) and the object must implement the #update method.

The attributes of the entry are:

  • id (integer/string)
  • target_channels (string)
  • created_at, updated_at (timestamps)
  • template (string)
@Luckeu Luckeu added the enhancement New feature or request label Jan 3, 2022
@Luckeu Luckeu self-assigned this Jan 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant