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

Remove "Recycle bin" feature #290

Open
6 tasks done
naveensingh opened this issue Jan 25, 2025 · 3 comments
Open
6 tasks done

Remove "Recycle bin" feature #290

naveensingh opened this issue Jan 25, 2025 · 3 comments
Labels
feature request Issue is about a new feature in the app

Comments

@naveensingh
Copy link
Member

naveensingh commented Jan 25, 2025

Checklist

  • I made sure that there are no existing issues - open or closed - to which I could contribute my information.
  • I made sure that there are no existing discussions - open or closed - to which I could contribute my information.
  • I have read the FAQs inside the app (Menu -> About -> FAQs) and my problem isn't listed.
  • I have taken the time to fill in all the required details. I understand that the request will be dismissed otherwise.
  • This issue contains only one feature request.
  • I have read and understood the contribution guidelines.

Feature description

Remove the recycle bin feature.

Why do you want this feature?

The feature should be removed because:

  • It only works with individual messages, not whole conversations.
  • Adding support for deleting conversations would turn this feature into the "Archive" feature and we'll have two things doing the same with different names.
  • Simpler code base, less potential for bugs.

Additional information

This might be a breaking change for some/many so it should probably be a part of version 2.0.0.

Related:

The archive feature is simple and enough for most use-cases, we could optionally add a 'Automatically delete archived messages after 30 days' option for those who want it to act like a recycle bin.

@naveensingh naveensingh added feature request Issue is about a new feature in the app needs triage Issue is not yet ready for PR authors to take up labels Jan 25, 2025
@naveensingh naveensingh removed the needs triage Issue is not yet ready for PR authors to take up label Jan 25, 2025
@s38b35M5
Copy link

s38b35M5 commented Jan 25, 2025

I would just ask to confirm that this message is correct before putting my full support behind this feature.

Disregard

@MGidnian
Copy link

Will it retain the functionality of "archiving" a single message and not the whole conversation?

I see how for some conversations you would like to completely archive/hide the conversation.
But another use case is: for many services/business - a common blocking use case it that you will get relevant messages (OTP, updates, etc) but will also get ads and other irrelevant messages in the same conversation.

To keep things clean and relevant - the best case scenario is to be able to archive/block/hide only certain messages but keep the rest in the inbox.

This functionality is supported today by the recycle bin, and also makes much sense for the keyword blocking mechanism in general.

@naveensingh
Copy link
Member Author

Will it retain the functionality of "archiving" a single message and not the whole conversation?

No but keyword blocking + #59 should solve such usecases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Issue is about a new feature in the app
Projects
None yet
Development

No branches or pull requests

3 participants