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

Implement Failed Notifications page #256

Open
Christoffer-Cortes opened this issue Aug 30, 2019 · 4 comments
Open

Implement Failed Notifications page #256

Christoffer-Cortes opened this issue Aug 30, 2019 · 4 comments
Labels
enhancement New or extended features.

Comments

@Christoffer-Cortes
Copy link
Contributor

Christoffer-Cortes commented Aug 30, 2019

Description

The failed notifications endpoint should be accessible through the GUI. There needs to be a page displaying the notifications of a specific subscription. Aggregated object snapshot and log message should be avaiable this way.

Note: This feature requires this eiffel-community/eiffel-intelligence#254 change in order to have access to the log message.

Motivation

We want to make it visible for the user if a subscription has failed to reach its notification meta.

Exemplification

Below are mock-up images of how the GUI could look like. The user would be able to sort subscriptions by failed notifications. The number of failed notifications would appear as a clickable button.
failed-notifications-11-subscription-page

When the button is clicked the failed notifications page opens with the listed failures for the specific subscription. The user can then click on aggregated object button to view the snapshot or the message button to view the log message. A download json data button would also be available to see the raw data.
failed-notifications-12-failed-page

There should also be a clear option to go back to the subscription page since this is a sub-page that is otherwise not reachable other than through the subscription table.
The simple option is the back button which could simply look like this:
failed-notifications-14-back
There is also the possiblity of adding a breadcrumb to the top of subscription page when a user clicks on failed notifications. The implementation of this is not entirely clear though. Should the breadcrumb stay when you go back to the parent? Should there be synching with breadcrumb and the url path e.g. URL: 'underwear/boxers/medium/, Breadcrumb: 'Underwear / Boxers / S'
failed-notifications-13-breadcrumb

Benefits

The failed notifications api is no longer obscured and can easily be viewed from the GUI. This would alert the users if something is wrong the notification meta. Perhaps the service is down or there was a spelling mistake in the meta etc.

Possible Drawbacks

@e-pettersson-ericsson
Copy link
Member

In the second picture, the message button refers to the logs/response saved by EI back-end which contains information about what actually went wrong. The name of the button I think can be discussed a bit more. Isn't logs better?

@Christoffer-Cortes
Copy link
Contributor Author

Sure, log sounds good. The layout, colors, naming etc. is up for change of course.

@e-pettersson-ericsson
Copy link
Member

e-pettersson-ericsson commented Nov 5, 2019

In back-end we currently only support a REST API endpoint given ONE subscription name. We can't find failed notifications for several subscriptions at the same time, see: eiffel-community/eiffel-intelligence#346

Before we can update the front-end, the back-end needs to support both use cases - if we go with the assumed layout described above.

@e-pettersson-ericsson
Copy link
Member

Eiffel Intelligence back-end has been updated to handle GET requests for failed notifications with multiple subscriptions, see PR 385 so it should be possible to start working on enabling this feature in the front-end.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New or extended features.
Projects
None yet
Development

No branches or pull requests

2 participants