@Daimona Quickly checking the ArticleTopicFiltersRegistry class, and I'm wondering, how would I go around requesting a message key for a single topic? Is it reasonable to just call ArticleTopicFiltersRegistry::getTopicMessages( [ $topicName ] )? Or should we have a more efficient way of doing single-topic queries?

Implementation would likely require adding a hard dependency on WikimediaMessages.

You could also fall back to an empty list of topics when WikimediaMessages is not available. That's what we're doing in CampaignEvents. The taxonomy that the topic list is based on is also kinda Wikipedia-specific, so I do believe it would be a good idea to reflect this in the code somehow.

In T386018#10536581, @Urbanecm_WMF wrote:

@Daimona Quickly checking the ArticleTopicFiltersRegistry class, and I'm wondering, how would I go around requesting a message key for a single topic? Is it reasonable to just call ArticleTopicFiltersRegistry::getTopicMessages( [ $topicName ] )? Or should we have a more efficient way of doing single-topic queries?

Yes, that is the intended way. We could add a shorthand if needed, I just thought the current implementation wasn't too inconvenient.

In T386018#10537625, @Daimona wrote:

Yes, that is the intended way. We could add a shorthand if needed, I just thought the current implementation wasn't too inconvenient.

I was mostly thinking about effectiveness, as that method appears to build something for all topics, only to select only one. If that is fine/expected, I don't think a shorthand is necessary.

In T386018#10546253, @Urbanecm_WMF wrote:

In T386018#10537625, @Daimona wrote:

Yes, that is the intended way. We could add a shorthand if needed, I just thought the current implementation wasn't too inconvenient.

I was mostly thinking about effectiveness, as that method appears to build something for all topics, only to select only one. If that is fine/expected, I don't think a shorthand is necessary.

Ah, I see. I think it's fine, the overhead should be minimal because the topic list is of fixed size and the code is basically just doing a for loop.

Change #1122954 had a related patch set uploaded (by Daimona Eaytoy; author: Daimona Eaytoy):

[integration/config@master] zuul: Make ContentTranslation and GrowthExperiments depend on WikimediaMessages

https://gerrit.wikimedia.org/r/1122954

Change #1122964 had a related patch set uploaded (by Daimona Eaytoy; author: Daimona Eaytoy):

[mediawiki/extensions/GrowthExperiments@master] Obtain article topic definitions from WikimediaMessages

https://gerrit.wikimedia.org/r/1122964

Change #1122954 merged by jenkins-bot:

[integration/config@master] Make ContentTranslation and GrowthExperiments depend on WikimediaMessages

https://gerrit.wikimedia.org/r/1122954

We're adding into our current Sprint and we'll post code review feedback soon, thanks for working on this @Daimona

Change #1129337 had a related patch set uploaded (by Sergio Gimeno; author: Sergio Gimeno):

[mediawiki/extensions/GrowthExperiments@master] feat(Topics): avoid hard dependency on WikimediaMessages for loading topics

https://gerrit.wikimedia.org/r/1129337

Change #1129337 abandoned by Sergio Gimeno:

[mediawiki/extensions/GrowthExperiments@master] feat(Topics): avoid hard dependency on WikimediaMessages for loading topics

Reason:

Squashed in Ie05689c803098bda461a6de26fb17b49f41239bb

https://gerrit.wikimedia.org/r/1129337

Change #1122964 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] Obtain article topic definitions from WikimediaMessages

https://gerrit.wikimedia.org/r/1122964