add subcomands for notifications #243

Closed
opened 1 year ago by 6543 · 5 comments
6543 commented 1 year ago
Collaborator

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 1 year ago
Collaborator

I would like to provide a PR for this?

I would like to provide a PR for this?
Collaborator

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 1 year ago
Poster
Collaborator

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 ...
Collaborator

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....
Collaborator

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 9 months ago
6543 modified the milestone from v0.8.0 to v0.9.0 4 months ago
6543 closed this issue 3 months ago
6543 referenced this issue from a commit 3 months ago
noerw modified the milestone from v0.9.0 to v0.8.0 2 months ago
Sign in to join this conversation.
Loading…
There is no content yet.