Missing Discourse Math plugin for translating?

[duy]

I saw this option in Meta Discourse’s rich text editor. I guess it’s Discourse Math, but I can’t find this plugin on Crowdin for translation.

Is this plugin missing on Crowdin?

P/S: And may I ask what plugin the small globe icon (“Post Language”) after the title belongs to?

[Moin]

The Math plugin is included in core, so you won’t find it as a separate project on Crowdin but within the Discourse project. I can see it there:

The problem is that when the client.en.yml was added in the update, no one updated the configuration to include that file in the translation process

translator.yml

[duy]

Well. I hope someone update the configuration to include that file, soon

[Moin]

If it is important for you, you can change the text there without waiting for the translation.
Customize text in Discourse works for texts that haven’t been translated like discourse_math.composer.insert_math too

[duy]

Thank you. That’s cool but I’ll wait.

[david (David Taylor)]

Thanks for the report @duy, and for the diagnosis @Moin. Here’s a PR to add it:
https://github.com/discourse/discourse/pull/37107

[duy]

Rapid and hard-work. That’s why I like Discourse Thank you @david !

[duy]

@david I think I’ve found some other strings that aren’t available for translation (or there are changed but they are not effective).

1. Footnote

2. Post Language button on editor

3. xxx posts were split to a new topic

4. This topic was automatically closed

5. Originally published at

6. post.in_reply_to

[Moin]

@duy

xxx posts were split to a new topic

That is move_posts.new_topic_moderator_post. I can find that in Crowdin

@duy

This topic was automatically closed

This one is topic_statuses.autoclosed_enabled_lastpost_days

I wonder if there is a problem with the plural versions

[david (David Taylor)]

@duy

1. Footnote

Make sure you’re changing the correct string. This one is js.footnote.add. There is a similar one under discourse-ai, but that’s used elsewhere.

@duy

2. Post Language button on editor

@keegan looks like we accidentally hardcoded this one?

frontend/discourse/app/components/post-language-selector.gjs

@duy

3. xxx posts were split to a new topic
4. This topic was automatically closed
5. Originally published at

These three are considered to be part of the post content. So the translation is only applied when the post (or ‘small action’) is created.

@duy

6. post.in_reply_to

Will be fixed by UX: Fix translation of post.in_reply_to by davidtaylorhq · Pull Request #37175 · discourse/discourse · GitHub. Thanks!

[Moin]

@david

Make sure you’re changing the correct string. This one is js.footnote.add.

I think the footnote client.yml is not available on Crowdin, just like math was not.

translator.yml

https://github.com/discourse/discourse/pull/37178

[duy]

@Moin

That is move_posts.new_topic_moderator_post. I can find that in Crowdin

Agreed. I found it.

@Moin

This one is topic_statuses.autoclosed_enabled_lastpost_days

I translated it two days ago, but it hasn’t been updated like the other strings.

@Moin

I wonder if there is a problem with the plural versions

There’s no problem with the plurals, I just can’t find them

@david

Make sure you’re changing the correct string. This one is js.footnote.add. There is a similar one under discourse-ai, but that’s used elsewhere.

Tried both. But I can’t found.

@david

These three are considered to be part of the post content. So the translation is only applied when the post (or ‘small action’) is created.

So, this means that only posts created after the translation is available will be translated?

[Moin]

@duy

So, this means that only posts created after the translation is available will be translated?

Yes. This is like a post you create; the content does not change because a translation is updated. Also, the post is created in the default language of the forum. So here on Meta, new small action posts about posts being split will still be in English.
It’s a little like the “Continuing the discussion from %{postLink}:” text that is added when you reply as a new topic. The text is part of the post. It’s not dynamic.

[Moin]

@duy

Tried both. But I can’t found.

You can try again now. My pull request to add the configuration for the translator bot was merged

[duy]

Thank you! This is good

[keegan (Keegan George)]

@david

@keegan looks like we accidentally hardcoded this one?

Ah yes! Fixed here:

https://github.com/discourse/discourse/commit/c9b80060329ac30f81c96cf3c5782d659ebce3dd

Thanks!