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.

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

Messaging API: deleted messages #5496

Open
mmd-osm opened this issue Jan 13, 2025 · 4 comments
Open

Messaging API: deleted messages #5496

mmd-osm opened this issue Jan 13, 2025 · 4 comments

Comments

@mmd-osm
Copy link
Contributor

mmd-osm commented Jan 13, 2025

A while back I've noticed that the Messaging API allows to retrieve deleted messages via the API (GET /api/0.6/user/messages/_nnn_). This seems to be inconsistent to how our UI works, where deleted messages are no longer shown.

We also don't have a concept of a Trash folder where deleted messages would be moved to and automatically (physically) deleted after some time, with the option for users to trigger a physical deletion at any time.

The Messaging API requirements section doesn't seem to cover the topic in sufficient detail. What would be a sensible approach?

@tomhughes
Copy link
Member

I think this was discussed during the PR review and it was decided to keep it like this?

@tomhughes
Copy link
Member

Yes #4605 (comment) is where I raised it.

@mmd-osm
Copy link
Contributor Author

mmd-osm commented Jan 13, 2025

Undeleting messages is fine as a use case, that’s how the trash folder could work. What I’m really missing is a way to permanently remove messages with no way to undelete them. Has this been discussed already at some point?

@tomhughes
Copy link
Member

Well the general rule is that we never delete anything for real in OSM so no.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants