-
Notifications
You must be signed in to change notification settings - Fork 167
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
Issue #3436013: Fix sidebar blocks display in flexible groups #3833
Issue #3436013: Fix sidebar blocks display in flexible groups #3833
Conversation
Thanks for contributing towards Open Social! A maintainer from the @goalgorilla/maintainers group might not review all changes from all teams/contributors. Please don't be discouraged if it takes a while. In the meantime, we have some automated checks running and it might be that you will see our comments with some tips or requests to speed up the review process. 😊 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @tregismoreira,
Changes look good to me on a clean test env, and was tested on existing env as well.
I put more details in a ticket.
Jut please rename hook update numbers, since we plan to release it in version 12.2.x.
bfcca36
to
9d33651
Compare
@ribel your comments are done, and the commits are cleaned. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update hooks were tested and configs are as expected.
51d1a7b
to
39c801f
Compare
Since these changes depends on version 13.0.0 I have changed milestone and update hooks to |
39c801f
to
542515e
Compare
To ensure the sidebar blocks will be properly displayed in flexible groups, this commit implements an update script to: - Delete the Blocks visibility settings, which was mistakenly set to be restricted to just a few group types; - Delete the Views' Contextual filter validation criteria, which was also mistakenly set to enable just a few group types. A Behat test was also provided to make sure all 3 sidebar blocks are present in the group.
542515e
to
bd27f89
Compare
Problem
We received a bug report saying that some of our clients can't see the sidebar blocks "Upcoming events" and "Newest topics" in flexible groups.
As a result of a extensive investigation, we discovered that the issue is caused by 3 misconfigurations:
social_course
module is overriding some of the Views configs affecting the query for those blocks.We also discovered that this issue does not happen to fresh installations.
Solution
Since the config overriding (3rd item mentioned above) is already been taken care on another issue (#3416433), we should write an update script to reset the other two configs: block visibility settings, and contextual filter validation criteria, so they will match with the install config state.
We should also write a Behat test to make sure the blocks appears properly in the flexible group sidebar region.
Issue tracker
Theme issue tracker
N/A
How to test
feature/3436013-fix-sidebar-blocks-display-in-flexible-groups
drush updb
)Expected result
We should be able to see all 3 group sidebar blocks:
Behat test expected result
Run the Behat test
tests/behat/features/capabilities/groups/flexible/groups-flexible-sidebar-blocks.feature
Definition of done
Before merge
After merge
Screenshots
Release notes
Ensure Events and Topics blocks are displayed in the group sidebar.
Change Record
Translations