Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Feature: Flag Mails as read #58

Closed
SpikePy opened this issue Nov 7, 2017 · 4 comments
Closed

Feature: Flag Mails as read #58

SpikePy opened this issue Nov 7, 2017 · 4 comments
Assignees
@jhillyerd
Labels
back end done feature front end
Projects
Modernization
Milestone
v2.0.0

Comments

@SpikePy
Copy link

SpikePy commented Nov 7, 2017

If you open a mailbox in the web interface to see all its mails there is no indicator on which mails have already been read.

It would be nice if you could add such a feature by adding an attribute "read" that equals to false/true OR add another class as indicator. Using a class you could also add a css-rule that prints the Header of an unread message bold.

Example with added class
@jhillyerd jhillyerd changed the title Feature-Request: Flag Mails as read Feature: Flag Mails as read Dec 16, 2017
@jhillyerd
Copy link
Collaborator

jhillyerd commented Dec 16, 2017 •
edited

  • Add seen flag to message interface, storage implementations.
  • Plumb seen flag into REST model/API.
  • Mark messages seen in Web UI.
  • Highlight unseen messages in Web UI.
  • Investigate REST API query for unseen.

@jhillyerd jhillyerd added this to the v1.2.1 milestone Dec 16, 2017
@jhillyerd
Copy link
Collaborator

Bumping to 1.3 since this would require a datastore wipe of existing installs.

@jhillyerd jhillyerd modified the milestones: v1.2.1, v1.3.0 Dec 30, 2017
@SpikePy
Copy link
Author

SpikePy commented Jan 17, 2018

Can you also extend the API with this flag, so that you are able to request for unread/read mails?

@jhillyerd jhillyerd added this to To do in Modernization Mar 25, 2018
@jhillyerd jhillyerd moved this from To do to In progress in Modernization Apr 1, 2018
@jhillyerd jhillyerd self-assigned this Apr 1, 2018
jhillyerd added a commit that referenced this issue Apr 1, 2018
@jhillyerd
storage: Add Seen flag, tests for #58
f25450f
jhillyerd added a commit that referenced this issue Apr 1, 2018
@jhillyerd
rest: Implement MarkSeen for #58
0d8088e 
- message: Add MarkSeen to Manager, StoreManager.
- rest: Add PATCH for /mailbox/name/id.
- rest: Add MailboxMarkSeenV1 handler.
- rest: Add Seen to model.
- rest: Update handlers to set Seen.
- rest: Add doJSONBody func.
jhillyerd added a commit that referenced this issue Apr 1, 2018
@jhillyerd
storage: Add Seen flag, tests for #58
cc5cd7f
jhillyerd added a commit that referenced this issue Apr 1, 2018
@jhillyerd
rest: Implement MarkSeen for #58
dc02092 
- message: Add MarkSeen to Manager, StoreManager.
- rest: Add PATCH for /mailbox/name/id.
- rest: Add MailboxMarkSeenV1 handler.
- rest: Add Seen to model.
- rest: Update handlers to set Seen.
- rest: Add doJSONBody func.
jhillyerd added a commit that referenced this issue Apr 1, 2018
@jhillyerd
ui: Mark messages as seen after 1.5s for #58
c695a26 
Embolden subject font for unseen messages.
@jhillyerd
Copy link
Collaborator

Looking at the REST API, it doesn't support any other query uses, so I don't see a reason to add seen. The message metadata is pretty small, and now contains a seen boolean.

@jhillyerd jhillyerd added the done label Apr 1, 2018
jhillyerd added a commit that referenced this issue Apr 1, 2018
@jhillyerd
Merge branch 'feature/read-58' for #58
0b3c18e
@jhillyerd jhillyerd moved this from In progress to Done in Modernization Apr 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jhillyerd jhillyerd

Labels
back end done feature front end
Projects
No open projects
Modernization
  
Done
Milestone
v2.0.0
Development

No branches or pull requests
2 participants
@jhillyerd @SpikePy