add subcomands for notifications #243

Closed
opened 2020-10-22 18:18:19 +00:00 by 6543 · 5 comments
Owner

move main feature from notif into notif ls|list and add subcomands pin & read, unread

move main feature from `notif` into `notif ls|list` and add subcomands `pin` & `read`, `unread`
6543 added the
kind
feature
label 2020-10-22 18:18:19 +00:00
Member

I would like to provide a PR for this?

I would like to provide a PR for this?
Member

Ok. Great I will start to work on that.

Ok. Great I will start to work on that.
noerw added the
status/has-pull
label 2020-12-08 22:51:58 +00:00
Author
Owner

the idear is to be not only able to filter & list

but also to mark notifications as read/unread/pinned ...

the idear is to be not only able to filter & list but also to mark notifications as read/unread/pinned ...
Member

So we will mark notifications as read by calling tea notif read... and unread via tea notif unread ... and pinned by calling tea notif pinned .....
Using tea notif ls... for listing them?

For tea notif ls .. we need supplemental options to difference between read, unread and pinned notifications...

Then I have misunderstood the orginal description....

So we will mark notifications as `read` by calling `tea notif read...` and `unread` via `tea notif unread ... ` and `pinned` by calling `tea notif pinned ..`... Using `tea notif ls... ` for listing them? For `tea notif ls ..` we need supplemental options to difference between `read`, `unread` and `pinned` notifications... Then I have misunderstood the orginal description....
Member

For tea notif ls .. we need supplemental options to difference between read, unread and pinned notifications...

Just add --state all|pinned|read|unread (with default pinned+read) to tea notif ls

But this seems optional to me:

  • Pinned notifications could be shown always on top of unread notifications
  • Read notifications are, well.. read, and as such not a notification anymore (I can see the situation of getting back to a notification that was dismissed by mistake, but that seems like a minor usecase..)
> For tea notif ls .. we need supplemental options to difference between read, unread and pinned notifications... Just add `--state all|pinned|read|unread` (with default pinned+read) to `tea notif ls` But this seems optional to me: - Pinned notifications could be shown always on top of unread notifications - Read notifications are, well.. read, and as such not a notification anymore (I can see the situation of getting back to a notification that was dismissed by mistake, but that seems like a minor usecase..)
noerw added this to the v0.8.0 milestone 2021-03-12 11:56:10 +00:00
6543 modified the milestone from v0.8.0 to v0.9.0 2021-08-16 12:54:32 +00:00
6543 closed this issue 2021-09-05 17:11:17 +00:00
noerw modified the milestone from v0.9.0 to v0.8.0 2021-09-22 21:36:42 +00:00
Sign in to join this conversation.
No Milestone
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: gitea/tea#243
No description provided.