feat: Add reason field to notification output #79
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
description
The
WebUI
also shows the user thenotification reason
1. In some situations, this may beuseful to the user. This PR would add the
reason
to the notification output.Points to consider:
reason
?type
.unread
notifications gets pushed further to the right and may not be visiblewith small terminals or extremely long repo names.
the full name is needed to perform action on it, for example opening it in the browser.
magenta
dot to the very beginning of the notification line orafter the timestamp.
Footnotes
REST API endpoints for notifications - GitHub Docs: Notification Reasons ↩