Problem/Motivation

Due to the issues reported here https://www.drupal.org/node/2850417 which in turn lead to the install issues here https://www.drupal.org/node/2881371 this module should be disabled immediately!

Proposed resolution

Remove the module from the install profile (there should be a patch here but I'm not setup currently to do this!) If someone else could please chip in I'd be eternally grateful!

Thanks to Kristiaan Van den Eynde for the advice!

Comments

Andy_D created an issue. See original summary.

jaapjan’s picture

jaapjan CreditAttribution: jaapjan at Open Social commented
Priority: Critical » Normal

Hi Andy,
thanks for the extensive information. I think it only happens in some scenarios - depending on the YamlParser used I guess - since I cant reproduce it on our Docker set-up or Platform.sh environments.

Anyway, you are right that this module should not be enabled anyway at this point. I've opened a pull request:
https://github.com/goalgorilla/open_social/pull/390

If you want feel free to test it with this patch file: https://patch-diff.githubusercontent.com/raw/goalgorilla/open_social/pul...

kristiaanvandeneynde’s picture

kristiaanvandeneynde
Primary language Dutch
Location Antwerp, Belgium
CreditAttribution: kristiaanvandeneynde at Deeson commented
Priority: Normal » Critical

The group role sync module will be empty as of #2883238: Move the grolesync module to the main module for outsiders only.. So it should no longer break any install. When I cut a new release, I'll mention it in the notes. There is absolutely no need to have it still enabled and it will be removed in 8.x-1.0. You have until then to make sure it is uninstalled everywhere.

kristiaanvandeneynde’s picture

kristiaanvandeneynde
Primary language Dutch
Location Antwerp, Belgium
CreditAttribution: kristiaanvandeneynde at Deeson commented
Priority: Critical » Normal
frankgraave’s picture

frankgraave CreditAttribution: frankgraave for Open Social commented
Status: Active » Fixed

  • frankgraave authored 1456c36 on 8.x-1.x 
    Merge pull request #390 from goalgorilla/2882738-grolesync

#2882738:...
  • jaapjan committed 98b4b8f on 8.x-1.x
    #2882738 by jaapjan: remove grolesync.
  • jaapjan committed d75c951 on 8.x-1.x
    Issue #2882738 by jaapjan: uninstall grolesync.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.