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

mark all as read except threads #14

Closed
wants to merge 1 commit into from

Conversation

grvn-ht
Copy link

@grvn-ht grvn-ht commented Apr 11, 2024

This Pull Request is meant to read all messages from users except bot before they connect to their account for first time in order not to flood synapse server with read requests.

@chagai95
Copy link

Is there a reason this is not merged? @HerHde should we test this?

@grvn-ht
Copy link
Author

grvn-ht commented Apr 17, 2024

Maybe replace condition:
if (message.md) {
This is meant to only select events that are messages
But after playing with migration I found that old messages did not have this "md" key
But don't know what could discriminate event messages other than this

@HerHde
Copy link
Collaborator

HerHde commented Apr 18, 2024

The reason from my side is just that the time I may allocate for this project is currently very limited. If you want to test it it helps, however.

Just a note, on first glance I wondered about the Markdown parts in this PR.

@HerHde HerHde self-assigned this Jun 17, 2024
@HerHde
Copy link
Collaborator

HerHde commented Jun 17, 2024

As I finally have a bit of time at hand, I took the liberty to merge and rewrite this.

But don't know what could discriminate event messages other than this

This lead me to use filters for m.room.message, while rewriting it to query Synapse instead of parsing the input file. And I put that together with the existing structure to check excess room members.

Have a look if the current main branch fulfils your needs, and if so, we can close #11

@HerHde HerHde closed this Jun 17, 2024
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.

3 participants