Here is initial patch for clean install, but update hook still needed.
ribel → created an issue.
Hi Bojan,
Please check the proposed changes in the applies() method.
First, we tried to use routeMatch service, but this approach was not working so I used pathValidator and routeProvider.
Also added a new testRouteOptOut() to the existing SimpleOauthAuthenticationTest.php.
I see that all checks pass and main PHPUnit run as well, but not sure about next minor and max PHP version (looks like unrelated to proposed changes).
Also tested with Collabora Online and it works as expected.
ribel → made their first commit to this issue’s fork.
Closing as not relevant to Distro.
Also see this one for other accessibility improvements:
https://www.drupal.org/project/social/issues/3503294
🐛
Accessibility improvements
Active
Changes were tested and tagged in version 6.2.2
The fix was merged and released in version 2.0.5
Thank you, for reporting and fixing this issue.
Changes were tested and merged
Changes were tested and looks good
Merged in PR: https://github.com/goalgorilla/open_social/pull/4250
Changes were tested and released in version 6.2.1
Fixed and merged in https://github.com/goalgorilla/open_social/pull/4249
Fixed and merged in https://github.com/goalgorilla/open_social/pull/4248
PR was tested and merged:
https://github.com/goalgorilla/open_social/pull/4205
Changes were tested and work as expected.
Changes were tested and look good.
Changes were tested and looks good
Changes were tested and looks good
Changes were tested and fixes issue with popup on admin pages.
Changes were tested and looks good to me.
Changes were tested and merged for the next release.
Changes were tested and works according to the acceptance criteria.
The change was tested and merged.
Changes were tested and fixed in version 2.0.4
Changes were tested and works correctly.
Changes were tested and released in version 2.0.3.
Thank you!
Changes were tested and it works correctly.
Changes were tested and works as expected.
LGTM
Final PR was merged and will be included in next release: https://github.com/goalgorilla/open_social/pull/4131
Merged and released in version 3.0.5
Changes have been tested and work as expected.
Changes have been tested and work as expected.
Changes have been tested and works as expected.
Re-tested and merged in https://github.com/goalgorilla/open_social/pull/4185
Related PR was tested and merged: https://github.com/goalgorilla/open_social/pull/4141
Changes have been tested and works as expected.
PR was tested and merged, will be in next relase.
Changes were tested and released in version 6.1.7
After testing with other extensions and update to Drupal 10.3 we found that PR still needs some work
It was reverted in https://github.com/goalgorilla/open_social/pull/4130 and will be re-created after fixing all issues.
Changes were tested and merged in this PR: https://github.com/goalgorilla/open_social/pull/4099
Will be released in version 13.0.0-alpha16
Changes were tested and worked as expected.
Changes were tested and works as expected. Will be included in the next update (13.0.0-alpha15)
LGTM
Changes were tested and looks good to me.
PR was merged and will be in the next release (13.0.0-alpha13)
PR with a fix was tested and merged: https://github.com/goalgorilla/open_social/pull/4076
Follow-up PR was merged and will be released in version 13.0.0-alpha13
Changes were tested and all work as expected.
Also for other Group bundles like Courses.
Changes were tested and released.
Changes were tested and works as expected.
Changes were tested and works as expected.
Changes were tested and looks good to me.
Changes were tested and looks good to me.
Changes looks good to me.
The changes looks good to me and was merged.
Changes were tested and work as expected now.
Changes were tested and worked as expected.
Changes were tested and worked as expected.
PR was merged and will be in the next release.
Changes were tested and worked as expected.
PR was merged and will be in the next release.
Changes were tested both on existing site and clean installation and works as expected now (also with Emails).
Changes were tested and merged
PR was approved and merged and will be in next release.
PR was merged and should be released in next version.
PR with a fix was tested and merged: https://github.com/goalgorilla/open_social/pull/4031
It will be in the next release.
PR was merged and will be in next release.
PR was tested and merged, will be in the next release.
PR was tested and merged, will be in the next release.
Changes were tested and works as expected. Will be released in next version.
PR was merged and should be relased in next version: https://github.com/goalgorilla/open_social/pull/3994
PR was merged and should be released in next version
PR was merged and will be included in next release: https://github.com/goalgorilla/open_social/pull/3964
Changes were tested and looks good.
Changes were tested and looks good
PR was tested, works as expected and without regressions.
Will be released in next update (13.0.0-alpha8)