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.

Sign up for GitHub

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

Jump to bottom

Group delete pushes with a delete-multiple for the talk clients as th… #939

Closed
wants to merge 1 commit into from
Closed

Group delete pushes with a delete-multiple for the talk clients as th… #939

wants to merge 1 commit into from

Conversation

nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Apr 22, 2021 •
edited
Loading

@nickvergessen nickvergessen added this to the Nextcloud 22 milestone Apr 22, 2021
@nickvergessen nickvergessen self-assigned this Apr 22, 2021
@nickvergessen nickvergessen mentioned this pull request Apr 30, 2021
Merged
@MorrisJobke MorrisJobke mentioned this pull request May 20, 2021
Merged
@MorrisJobke
Copy link
Member

🏓

@nickvergessen
Copy link
Member Author

Not too urgent, we can move it to 23 once the milestone exists

@MorrisJobke
Copy link
Member

Done.

@skjnldsv skjnldsv mentioned this pull request Oct 25, 2021
Merged
25 tasks
@nickvergessen nickvergessen removed this from the Nextcloud 23 milestone Oct 25, 2021
@blizzz blizzz added this to the Nextcloud 25 milestone Apr 21, 2022
@nickvergessen
Copy link
Member Author

@Ivansss did you implement this in the meantime?

@Ivansss
Copy link
Member

Ivansss commented Jul 11, 2022

No, it is not implemented.

This was referenced Aug 12, 2022
Merged
Merged
This was referenced Aug 24, 2022
Merged
Merged
This was referenced Sep 6, 2022
Merged
Merged
@skjnldsv skjnldsv mentioned this pull request Sep 15, 2022
Merged
This was referenced Sep 20, 2022
Closed
Merged
@blizzz blizzz modified the milestones: Nextcloud 25, Nextcloud 26 Sep 22, 2022
@SystemKeeper SystemKeeper mentioned this pull request Oct 26, 2022
Merged
@SystemKeeper
Copy link

@nickvergessen Implemented in nextcloud/talk-ios#953

@nickvergessen
Copy link
Member Author

Okay, then let me add this back to my todo

@nickvergessen nickvergessen mentioned this pull request Jan 25, 2023
Merged
@nickvergessen
Copy link
Member Author

Too many changes and conflicts, started to redo it in #1429
Leaving this old PR here so I can compare the steps

@nickvergessen nickvergessen deleted the bugfix/noid/reduce-delete-push-spam branch January 25, 2023 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nickvergessen nickvergessen

Labels
2. developing enhancement feature: push 📲
Projects
None yet
Milestone
Nextcloud 26
Development

Successfully merging this pull request may close these issues.

Combine and limit delete-notification pushes
5 participants
@nickvergessen @MorrisJobke @Ivansss @SystemKeeper @blizzz