Additional groups not visible on /about page

I upgraded to Discourse 3.5.0.beta6-dev (da0b303568) this morning.

After the update I was met with a red banner across the top of the site saying:

Admin notice: you’re using the discourse-add-groups-to-about theme component. This feature is now available in Discourse core. You should remove this theme component.

I did as I was told and deleted the component: Add groups to the /about page

I then went to my Discourse site settings and re-applied all the about page settings that I used in the old component:

But none of the groups are appearing on the about page:

Here’s how it used to look, before I deleted the component:

I’ve tried viewing in Safe Mode too, but no difference.

I think the show_additional_about_groups site setting is still hidden

But you need to enable that for the groups to show up.

4 Likes

Oh no :confused:

I can’t re-install the theme now because the big red warning banner is visible to all users, even people who are not logged in :cry:

1 Like

If you are self-hosted, you can enable the setting

5 Likes

Thanks @Moin

Do I have to do this manually or will this bug likely be fixed in a future update? :thinking:

I think a migration which was added 3 days ago should have enabled that setting. DEV: Migrate add-groups-to-about component settings to site settings by Drenmi · Pull Request #32910 · discourse/discourse · GitHub So , no idea if that will still happen on your forum and why it didn’t happen when you updated :woman_shrugging:

4 Likes

Great insight, thanks :slight_smile:

Is the show_additional_about_groups setting visible in your admin panel? :thinking:

No, but I haven’t installed the component. Maybe that’s why the migration didn’t happen for me. But I wasn’t really interested because I don’t use the feature.

In fact, I follow the PRs on GitHub mainly because the screenshots help with translation, especially for features like this that are not available in the interface at first.

1 Like

But if this is now in core, isn’t the option available to everyone now? :thinking:

It’s in core, but the show_additional_about_groups setting is still a hidden site setting, so it’s not visible.
I think the migration enabled it for sites which used the component before. I am not sure if it will be enabled for everyone or simply be removed. I think it is only needed during the migration process.

If I have understood correctly, the idea is that the show_additional_about_groups setting is added to core. Then a check is added to the theme component so that the groups are only displayed if show_additional_about_groups is false. This means that it simply continues to work as before. Afterwards everything is built in core, but only displayed if show_additional_about_groups is true.
After the settings have been automatically migrated, show_additional_about_groups is activated. Now the code from core shows the groups and no longer the component.
This migration was added 3 days ago. Then 2 days ago the notice was added in the component that was displayed to you.
I don’t know why the migration obviously didn’t work for you (neither the settings were automatically applied, nor was show_additional_about_groups activated).

I think it is definitely an oversight that the global warning does not check whether the user is an admin.

2 Likes

Thanks for the insight @Moin - very useful.

Hmm, so I guess that’s on for everyone by default then, everyone that wasn’t previously using the theme component?

And mine is simply not displaying any of the groups on the About page because my setting is stuck on false in the background still?

You wouldn’t believe how many emails and queries we’ve had from our members about this warning banner :see_no_evil_monkey:

1 Like

@Moin I’ve applied this via the rails console:

SiteSetting.show_additional_about_groups = true

And the extra groups have now re-appeared.

Thanks again for all your help and insight :person_bowing:

4 Likes