Remove broadcast source error sound

[loepare]

When being contributor to a broadcast, a nasty error sound occurs when a source error happens.
I thought for months that my headset was broken, because I couldn't match the sound to Lichess (I often have broadcasts open in the background). The personal story is not so important here, what I want to say is; I don't see the point of that sound, it doesn't really help much IMO.

[allanjoseph98]

lila/ui/analyse/src/study/relay/relayCtrl.ts

Lines 160 to 162 in c664cd0

	if (event.error) {
	if (this.data.sync.log.slice(-3).every(e => e.error)) site.sound.play('error');
	console.warn(`relay synchronisation error: ${event.error}`);

Plays after 3 consecutive sync errors. Don't think the console warning is useful either.

[SergioGlorias]

The sound even has a reason: if the broadcaster has broadcast open, he will be alerted that there is an error.

It helps a lot to warn that the broadcaster has to act and not stay asleep

Although it currently does not warn of errors like it used to (missing or disordered chapters) which was more important

[loepare]

I understand the intention behind it, but I think most people wouldn't even understand that this sound is caused by the error of a lichess source.

[SergioGlorias]

Note: this error only happens to those who are contributing to the round and not a global sound

[ornicar]

So? Keep or remove?

[SergioGlorias]

For me, keep
For admins the warnings seem useless since they can see the logs via IRC, but there are several people who do not have access and a warning is enough to solve the problem.

[SergioGlorias]

Plays after 3 consecutive sync errors. Don't think the console warning is useful either.

Previously, there was sound for any error, but it was limited because there were momentary errors