- Issue created by @xrobp
- Merge request !21Issue #3471105: Use correct group type id prefix in GroupContentRespectUnpublished views filter plugin query. → (Open) created by xrobp
- Status changed to Needs review
9 months ago 11:28pm 29 August 2024 - Status changed to RTBC
6 months ago 2:19pm 5 December 2024 - First commit to issue fork.
- 🇧🇪Belgium msnassar
For now we should ignore the failing tests in D11 builds. This is going to be fixed in another issue.
The tests using D10 are passing, this demonstrate that our tests are not good. So in order to demonstrate this issue, lets write a test that fails .
BTW, from group 2, I think we might not need this views plugin anymore! But lets demonstrate this later or in another ticket. - 🇧🇪Belgium msnassar
Now we have a failing tests... Lets have them passing by fixing the reported issue.
- 🇧🇪Belgium msnassar
I have cherry picked the fix from MR21. Now the tests are green again...
I think the exiting code in this test can be improved... Anyway, the next step is to demonstrate that we might not need this views plugin anymore!
- Merge request !28Issue #3471105: Use correct group type id prefix in GroupContentRespectUnpublished views filter plugin query → (Merged) created by msnassar
- 🇧🇪Belgium keszthelyi Brussels
Tested the fix in MR!28 and it solved the issue +1
- 🇧🇪Belgium msnassar
msnassar → changed the visibility of the branch issue/3471105-moderated-content-view to hidden.
- 🇧🇪Belgium msnassar
msnassar → changed the visibility of the branch 3471105-moderated-content-view-test-v2 to hidden.
- 🇧🇪Belgium msnassar
msnassar → changed the visibility of the branch 3471105-moderated-content-view-test-v2 to active.
-
hernani →
committed 372e963f on 3.0.x authored by
msnassar →
Issue #3471105 by msnassar, xrobp, slydevil, andras_szilagyi, keszthelyi...
-
hernani →
committed 372e963f on 3.0.x authored by
msnassar →
-
hernani →
committed cdd55cf5 on 2.0.x authored by
msnassar →
Issue #3471105 by msnassar, xrobp, slydevil, andras_szilagyi, keszthelyi...
-
hernani →
committed cdd55cf5 on 2.0.x authored by
msnassar →
Automatically closed - issue fixed for 2 weeks with no activity.