Is this intentional to not reflect from the list whether they are in an open or closed state?
I was looking into this recently because of #2261. The bookmark model does not contain state so it just knows the notificationType.
I think changing this to match with notificationModel is a good idea.
Sent with GitHawk
The complication here is that you will have to fetch data to get the latest status. Pretty big change. Might want to do this alongside a larger bookmarks refactor.
Sent with GitHawk
Just making a note that this also results in edited titles not dynamically reflecting
Good point. I want tot totally refactor how we do bookmarks...
Sent with GitHawk
I’m sure I’m missing a few but at least bookmark items I’ve, for lack of a better word, bookmarked 🤷♂️
Just aggregating everything for ease when project plan is laid out-
In addition to those from this post :
Most helpful comment
I was looking into this recently because of #2261. The bookmark model does not contain state so it just knows the notificationType.
I think changing this to match with notificationModel is a good idea.
Sent with GitHawk