might want to think about notifying all users when someone tunes in

This commit is contained in:
talksik
2022-05-06 10:44:38 -05:00
parent f3cd9edc95
commit 886f10cf23
2 changed files with 4 additions and 7 deletions
@@ -69,14 +69,10 @@ export function LineDataProvider({ children }) {
$ws.on( $ws.on(
SocketChannels.SOMEONE_CONNECTED_TO_LINE, SocketChannels.SOMEONE_CONNECTED_TO_LINE,
(res: SomeoneConnected) => { (res: SomeoneConnected) => {
toast( // TODO: change the correct masterLineData to contain this
`another user (${res.userId}) connected into line ${res.lineId} that you are in also connected to`
);
// change the correct masterLineData to contain this
console.log( console.log(
`here are all of the users in the tuned in room`, `connected to line...here are all of the users in the conected line ${res.lineId}`,
res.allConnectedIntoUserIds res.allConnectedIntoUserIds
); );
} }
@@ -84,6 +80,7 @@ export function LineDataProvider({ children }) {
// whether toggle tuned or temporarily // whether toggle tuned or temporarily
$ws.on(SocketChannels.SOMEONE_TUNED_TO_LINE, (res: SomeoneTuned) => { $ws.on(SocketChannels.SOMEONE_TUNED_TO_LINE, (res: SomeoneTuned) => {
// TODO: change the correct master line data to show user who's tuned in
toast( toast(
`another user (${res.userId}) tuned into line ${res.lineId} that you are in also` `another user (${res.userId}) tuned into line ${res.lineId} that you are in also`
); );
@@ -87,7 +87,7 @@ export default function LineDetailsTerminal() {
hover:scale-105 transition-all ${ hover:scale-105 transition-all ${
selectedLine.isUserBroadcasting || selectedLine.isOtherBroadcasting selectedLine.isUserBroadcasting || selectedLine.isOtherBroadcasting
? "bg-teal-800 text-white" ? "bg-teal-800 text-white"
: "text-teal-800 border-teal-800" : "text-teal-800 border-teal-800 border"
}`} }`}
> >
<FiSun className="text-lg" /> <FiSun className="text-lg" />