fixes concurrent rolling of same user with different name
All checks were successful
CI / deploy (push) Successful in 4m1s
CI / deploy (pull_request) Successful in 4m1s

This commit is contained in:
Arindy 2025-02-10 02:26:54 +01:00
parent f616726aae
commit 70f0cc99fc
2 changed files with 2 additions and 4 deletions

View File

@ -76,8 +76,6 @@
} )) } ))
} }
diceBox.roll(data.roll, { theme: data.theme?.length > 0 ? data.theme : 'default', themeColor: data.themeColor.length > 0 ? data.themeColor : '#4545FF' }); diceBox.roll(data.roll, { theme: data.theme?.length > 0 ? data.theme : 'default', themeColor: data.themeColor.length > 0 ? data.themeColor : '#4545FF' });
}) })
}) })
</script> </script>

View File

@ -14,8 +14,8 @@
const evtSource = new EventSource(url() + '/dice/{room}/results'); const evtSource = new EventSource(url() + '/dice/{room}/results');
evtSource.addEventListener('message', function (event) { evtSource.addEventListener('message', function (event) {
let data = JSON.parse(event.data); let data = JSON.parse(event.data);
let name = document.getElementById(data.user) ?? document.createElement('div'); let name = document.getElementById(data.user + '-' + data.name) ?? document.createElement('div');
name.id = data.user; name.id = data.user + '-' + data.name;
name.replaceChildren(...[]); name.replaceChildren(...[]);
let node = document.createElement('p'); let node = document.createElement('p');
let resultText = '' let resultText = ''