Featured committer displaying wrong information on eclipse.org homepage

The eclipse.org redesign seems to have changed how the Featured Committer section of the eclipse.org homepage pulls its content. It looks like it is set up to always describe the committer as a "Committer to (featured project of the month)" when it should be displaying the information we enter in the newsletter Edition in Newsroom.

added Bug prioritymedium severitynormal statetodo labels

assigned to @oliviergoulet

added team:webdev label

I'll poke around and see what could have caused this regression @jharrisj1m.

I took a quick look at an older version of eclipse.org, and the same Featured Committer and Featured Project appear:

The redesign uses the same endpoint as what we had before:

https://newsroom.eclipse.org/api/edition?options[orderby][nid]=DESC&pagesize=1

I also see Thomas Mader in the newsletter: https://newsroom.eclipse.org/eclipse-newsletter/2023/october/committer-profile-thomas-m%C3%A4der

Who is it that you expect to see instead, @jharrisj1m?

/cc @epoirier

I'll let @jharrisj1m confirm but I think what he means is to use the body value from the api.

https://newsroom.eclipse.org/api/edition

Rather then building the string yourself using the project name and committer status.

Oh, I see what @jharrisj1m and you are saying. Yeah, the widget doesn't use the API text. I will make that change.

To clarify, the featured project is displaying as it should, but the featured committer is showing the correct name, but the wrong description, as it should be what we enter into newsroom. The screenshot of the old eclipse.org shows how it should be appearing:

added statewip label and removed statetodo label

mentioned in merge request !122 (merged)

Here is a preview of the fix: https://preview-122--www-eclipse-org.eclipsecontent.org/home/

@jharrisj1m

Perfect, +1

@oliviergoulet will this be live this week?

Yes, I'll push live tomorrow. We had a security concern with how we receive and use the data.

Actually, I'm doing it now.

mentioned in commit 5fd3820d

closed with merge request !122 (merged)

added statedone label and removed statewip label