-
Notifications
You must be signed in to change notification settings - Fork 45
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
Comments
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? |
Sure, log sounds good. The layout, colors, naming etc. is up for change of course. |
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. |
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. |
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.
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.
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:
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'
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
The text was updated successfully, but these errors were encountered: