קבוצות נוספות לא נראות בדף /about

[Richie (Richie Rich)]

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.

[Moin]

I think the show_additional_about_groups site setting is still hidden

config/site_settings.yml

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

[Richie (Richie Rich)]

@Moin

the show_additional_about_groups site setting is still hidden

Oh no

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

[Moin]

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

Using Hidden Site Settings

This is a how-to guide about Hidden Site Settings, how to enable them, and why you might want to adjust them. Required user level: Administrator Important: Console access is required for some steps. In Discourse, hidden site settings refer to configuration options that are not readily visible or accessible through the standard admin dashboard interface. These settings are part of the backend configuration that can control various advanced or potent…

[Richie (Richie Rich)]

Thanks @Moin

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

[Moin]

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

[Richie (Richie Rich)]

Great insight, thanks

Is the show_additional_about_groups setting visible in your admin panel?

[Moin]

לא, אבל לא התקנתי את הרכיב. אולי בגלל זה המעבר לא קרה לי. אבל לא ממש התעניינתי כי אני לא משתמש בתכונה.

למעשה, אני עוקב אחרי ה-PR-ים בגיטהאב בעיקר בגלל שצילומי המסך עוזרים בתרגום, במיוחד לתכונות כמו זו שאינן זמינות בממשק בהתחלה.

[Richie (Richie Rich)]

@Moin

No, but I haven’t installed the component

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

[Moin]

זה נמצא בליבה, אבל ההגדרה show_additional_about_groups היא עדיין הגדרת אתר חבויה, כך שהיא לא גלויה.
אני חושב שההעברה הפעילה אותה עבור אתרים שהשתמשו ברכיב בעבר. אני לא בטוח אם היא תופעל עבור כולם או פשוט תוסר. אני חושב שהיא נחוצה רק במהלך תהליך ההעברה.

אם הבנתי נכון, הרעיון הוא שההגדרה show_additional_about_groups מתווספת לליבה. לאחר מכן מתווספת בדיקה לרכיב הנושא כך שהקבוצות יוצגו רק אם show_additional_about_groups הוא שקר (false). זה אומר שזה פשוט ממשיך לעבוד כמו קודם. לאחר מכן הכל בנוי בליבה, אבל מוצג רק אם show_additional_about_groups הוא אמת (true).
לאחר שההגדרות הועברו אוטומטית, show_additional_about_groups מופעל. כעת הקוד מהליבה מציג את הקבוצות ולא עוד הרכיב.
ההעברה הזו נוספה לפני 3 ימים. לאחר מכן, לפני יומיים, ההודעה נוספה ברכיב שהוצג לך.
אני לא יודע למה ההעברה כנראה לא עבדה בשבילך (גם ההגדרות לא הוחלו אוטומטית, וגם show_additional_about_groups לא הופעל).

אני חושב שזו בהחלט טעות שהאזהרה הגלובלית לא בודקת אם המשתמש הוא מנהל מערכת.

[Richie (Richie Rich)]

Thanks for the insight @Moin - very useful.

@Moin

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.

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?

@Moin

an oversight that the global warning does not check whether the user is an admin.

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

[Richie (Richie Rich)]

@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

[awesomerobot (Kris)]

@Richie

the big red warning banner is visible to all users

thanks for pointing it out, I’m going to get this fixed so others won’t run into this

[sam (Sam Saffron)]

רגע, האם נשארה כאן תקלה לא פתורה או שזה רק נושא #תמיכה בשלב זה?

[Moin]

חשבתי שההעברה הייתה אמורה להעביר את התצורה של @Richie מהרכיב להגדרות הליבה וגם הייתה אמורה להפעיל את הגדרת האתר הנסתרת show_additional_about_groups. סיבה לכך שזה לא עבד יכולה להיות שהוא שינה את השם, כי זה קל לעשות בממשק.
האם יש סיבה לכך שההעברה מסתמכת רק על השם במקום לבדוק גם רכיב שבו ה-remote_url מטבלת remote_themes תואם? זה היה תופס רכיבים ששמם שונה כל עוד הם הותקנו מהמאגר הרשמי.

הגדרת האתר הנסתרת, שלא הופעלה מכיוון שההעברה לא קרתה, מנעה מהליבה להציג את הקבוצות, ובמקום זאת, הרכיב עדיין הציג את הקבוצות. אבל ההודעה הגלובלית מהרכיב אמרה לו למחוק את הרכיב. אז הליבה עדיין לא הציגה את הקבוצות כי show_additional_about_groups עדיין היה מושבת, ולא היה קל להפעיל אותו.

אז אם ההעברה האוטומטית לא עבדה, איך מנהלים מעבירים ידנית? העתקת התצורה אינה הבעיה. אבל מתי הם עוברים בין הרכיב שמציג את הקבוצות לליבה שמציגה את הקבוצות מבלי להצטרך להפעיל את ההגדרה הנסתרת?
אולי היה עדיף להפעיל את show_additional_about_groups לכולם או להציג אותה בממשק לפני הוספת הודעה לרכיב שיש למחוק אותו. אז העברה ידנית הייתה עובדת, והליבה הייתה מציגה את הקבוצות לאחר מכן, כך שמחיקת הרכיב לא הייתה משאירה מנהלים בלי קבוצות בדף האודות.

נכון לעכשיו, כאשר מנהל מוסיף קבוצות להגדרת האתר about page extra groups, שום דבר לא קורה מכיוון שההגדרה הנסתרת show_additional_about_groups אינה מופעלת. זה מרגיש כמו באג, למרות שמנקודת מבטו של מפתח, הכל עובד כמצופה. אני חושב שיהיה קל יותר למנהל להבין מה קורה אם ההגדרה תהיה גלויה במקום נסתרת.

[Richie (Richie Rich)]

@Moin has summed up everything perfectly.

None of my settings were copied across, and when I did copy them the groups were still not visible on the /about page because the hidden setting prevented them from appearing and I had to use the Rails console to resolve the issue.

This is how I got stuck:

@Moin

when an admin adds groups to the about page extra groups site setting, nothing happens because the hidden show_additional_about_groups setting isn’t enabled

And this is the eternal question

@Moin

That feels like a bug, even though from a developer’s perspective, everything is working as expected.

[sam (Sam Saffron)]

@ted / @kris.kotlarek / @hugh any thoughts here? Does the migration Ted made need a refinement?

@Moin

Right now, when an admin adds groups to the about page extra groups site setting, nothing happens because the hidden show_additional_about_groups setting isn’t enabled. That feels like a bug, even though from a developer’s perspective, everything is working as expected. I think it would be easier for an admin to understand what happens if the setting were visible instead of hidden.

This feels very weird, why not simply delete the show_additional_about_groups site setting and just have it be true?

One less setting to worry about and the setting about page extra groups will always work as expected.

[ted (Ted Johansson)]

Hello, @Richie, @Moin!

We used this hidden site setting in the process of porting over the theme component. Due to a problem with the migration it wasn’t enabled correctly when the data was migrated.

Now that the port is done, we don’t really need this setting any more as @sam pointed out. We can simply look at the list of configured groups.

There’s a PR that does that here:

DEV: Have about page extra groups depend on groups selected

[Moin]

מעולה! חשבתי שזה יצטרך להישאר עד לגרסה היציבה הבאה כדי לוודא שהרכיב לא מציג את הקבוצות בנוסף לליבה עבור אלה שמשדרגים אז.

[Richie (Richie Rich)]

Thanks for the update @ted