Skip to content
This repository has been archived by the owner. It is now read-only.

When you reply to a user message, the original should be marked as read #4300

Closed
openstreetmap-trac opened this issue Jul 23, 2021 · 5 comments
Closed

Comments

@openstreetmap-trac
Copy link

@openstreetmap-trac openstreetmap-trac commented Jul 23, 2021

Reporter: Andy Allan
[Submitted to the original trac issue database at 12.15pm, Friday, 16th March 2012]

When I click on a message reply link in an email, and send a reply, it's almost certain that by that point I've read the message. I think it makes sense to automatically mark the message as read, rather than having to do that separately.

@openstreetmap-trac
Copy link
Author

@openstreetmap-trac openstreetmap-trac commented Jul 23, 2021

Author: TomH
[Added to the original trac issue at 3.18pm, Friday, 16th March 2012]

Fixed - will be in the next deployment.

@openstreetmap-trac
Copy link
Author

@openstreetmap-trac openstreetmap-trac commented Jul 23, 2021

Author: pnorman
[Added to the original trac issue at 7.20pm, Sunday, 2nd September 2012]

It appears to not be fixed - they're still not getting marked as read.

@openstreetmap-trac
Copy link
Author

@openstreetmap-trac openstreetmap-trac commented Jul 23, 2021

Author: TomH
[Added to the original trac issue at 7.28pm, Sunday, 2nd September 2012]

Yes it is fixed.

@openstreetmap-trac
Copy link
Author

@openstreetmap-trac openstreetmap-trac commented Jul 23, 2021

Author: TomH
[Added to the original trac issue at 11.51pm, Sunday, 2nd September 2012]

I misread this ticket and thought it was talking about the case where you reply to the message by sending an email - that is what I fixed.

The case of clicking on the reply link, which is what the ticket is actually about, does still need dealing with.

@openstreetmap-trac
Copy link
Author

@openstreetmap-trac openstreetmap-trac commented Jul 23, 2021

Author: TomH
[Added to the original trac issue at 8.07am, Monday, 3rd September 2012]

The "replying via the web" case is now fixes as well.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant