The alert dialog depends on the warning text. That should be changed as well.

In addition to Allan Joseph's comment, please consider if it would now be suitable to make this into a translateable (i18n) string. In terms of text, every word has been carefully considered, so the content can be considered final.

Beyond the usual requirements of adding a translateable string (documented on the Lila wiki), you would also need to ensure that the alert dialog still triggers if it an i18n string.

The alert dialog depends on the warning text. That should be changed as well.

lila/ui/round/src/round.ts

Line 126 in 1921093

if (line.u === 'lichess' && line.t.toLowerCase().startsWith(`warning, ${myUserId()}`))

Thanks! should be fixed. It's difficult to test considering direct challenges do not trigger it.

In addition to Allan Joseph's comment, please consider if it would now be suitable to make this into a translateable (i18n) string. In terms of text, every word has been carefully considered, so the content can be considered final.

Beyond the usual requirements of adding a translateable string (documented on the Lila wiki), you would also need to ensure that the alert dialog still triggers if it an i18n string.

I consider translation a separate feature, and it's outside of the scope of that PR.