- Issue created by @anjaliprasannan
- Merge request !11017Issue #3502540: Add date filter for moderated content. → (Open) created by Unnamed author
- 🇳🇿New Zealand quietone
Changes are made on on 11.x (our main development branch) first, and are then back ported as needed according to the Core change policies → .
- 🇺🇸United States smustgrave
@anjaliprasannan I would recommend checking the MR before moving to view
Can see the MR is showing 1000s of files and not mergable.
Also the issue summary is incomplete.
Haven't reviewed but will need test coverage if not added.
- Merge request !11034#3502540: Add date filter to moderated content filter. → (Open) created by Unnamed author
- 🇮🇳India anjaliprasannan
anjaliprasannan → changed the visibility of the branch 3502540-add-date-range to hidden.
- 🇮🇳India sdhruvi5142
Hi,
I've applied the MR!11034 successfully for drupal core 11.x and it is working as expected.Steps I followed for testing:
1. Install Content Moderation module on my local
2. Go to Content > Created few content with Moderate state
3. Go to Content > Moderate Content and observe the changes there.Testing Result:
Here after applying the patch, able to see the date filter and its functionality is working properly as expected.Suggestion:
I would like to share one suggestion here that I can see there is no space between the Filter button and Authored On Filter. According to me there should be space between them. Due to this issue keeping in need review state for further review around this point. ThanksAttaching Screenshots for reference
Status - Needs Review - 🇮🇳India anjaliprasannan
@sdhruvi5142 added space before filter action button. Please review.
- 🇺🇸United States smustgrave
Updating claro seems out of scope. Also on other views core ships we don't expose the operator.
For me this seems like a unique need so I may be a -1 for adding to core.