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

Skip archived messages in deleteMessages.js #392

Closed
wants to merge 1 commit into from
Closed

Skip archived messages in deleteMessages.js #392

wants to merge 1 commit into from

Conversation

dylif
Copy link

@dylif dylif commented Jun 22, 2022

This is a simple fix to continually skip archived messages when they are found in search. However, it does not prevent the archived messages from appearing in a search. Regardless, this partially fixes #242 as archived messages will be automatically skipped and not waste time.

@negswell
Copy link

negswell commented Jun 25, 2022

@dylif this works perfectly , i wrote the code for the full solution which prevents the api call, just testing it and will make a pr for it soon.

@dylif
Copy link
Author

dylif commented Jun 26, 2022

@negswell Great. Thanks for doing that, it's even better. Shall I close with PR then?

@negswell
Copy link

@dylif you could , i have raised a pr for the fix now.

@dylif
Copy link
Author

dylif commented Jun 27, 2022

#393 fixes this issue better than I have. Please direct all attention there.

@dylif dylif closed this Jun 27, 2022
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

Successfully merging this pull request may close these issues.

[Feature request] Archived threads need to be automatically skipped
2 participants