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

Mailing list batching #227

Merged
merged 1 commit into from
Jan 13, 2025
Merged

Mailing list batching #227

merged 1 commit into from
Jan 13, 2025

Conversation

reinhardt
Copy link
Contributor

@reinhardt reinhardt commented Jan 10, 2024

@thet
Copy link
Member

thet commented Jan 10, 2025

@reinhardt awesome! exactly a year after you submitted this PR I again work on this issue and can directly use this PR.
I started my own branch with implementing only the page limit part, but your PR is more complete and better.
❤️

@thet thet marked this pull request as ready for review January 13, 2025 08:41
@thet thet force-pushed the scrum-1638-mailing-list-batching branch 2 times, most recently from c1af1a9 to cda74c5 Compare January 13, 2025 10:40
@reinhardt
Copy link
Contributor Author

The test failure only seems to be caused by a differing sorting. Do you want me to look into it?

@thet
Copy link
Member

thet commented Jan 13, 2025

@reinhardt yes please...

that's funny - a different sorting in plone 5.2 and 6.0.
maybe we can even give up the 5.2 tests? do we still have non 6.0 osha.oira deployments?

@reinhardt
Copy link
Contributor Author

Good point - I didn't notice the failures are for 5.2 only. Osha is on 6.0, so feel free to drop the 5.2 tests!

@reinhardt reinhardt mentioned this pull request Jan 13, 2025
@thet
Copy link
Member

thet commented Jan 13, 2025

@reinhardt done here: #300

@thet thet force-pushed the scrum-1638-mailing-list-batching branch from cda74c5 to bd97720 Compare January 13, 2025 14:05
@reinhardt reinhardt merged commit 7ab8dff into main Jan 13, 2025
1 check passed
@reinhardt reinhardt deleted the scrum-1638-mailing-list-batching branch January 13, 2025 14:16
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.

2 participants