Skip to content

Commit

Permalink
fix: Do not show notification when call feedback is skipped
Browse files Browse the repository at this point in the history
  • Loading branch information
thisisamir98 committed Dec 19, 2024
1 parent 341b898 commit efc6e22
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ export const QualityFeedbackModal = () => {
return null;
}

const handleCloseModal = () => {
const handleCloseModal = (skipNotification = false) => {
if (!selfUser) {
setQualityFeedbackModalShown(false);
return;
Expand All @@ -80,7 +80,10 @@ export const QualityFeedbackModal = () => {

currentStorageData[selfUser.id] = isChecked ? null : dateUntilShowModal.getTime();
localStorage.setItem(CALL_QUALITY_FEEDBACK_KEY, JSON.stringify(currentStorageData));
submittedNotification.show();

if (!skipNotification) {
submittedNotification.show();
}
} catch (error) {
logger.warn(`Can't send feedback: ${(error as Error).message}`);
} finally {
Expand All @@ -102,7 +105,7 @@ export const QualityFeedbackModal = () => {
[Segmentation.CALL.QUALITY_REVIEW_LABEL]: RatingListLabel.DISMISSED,
});

handleCloseModal();
handleCloseModal(true);
};

return (
Expand Down
2 changes: 1 addition & 1 deletion src/types/i18n.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1630,7 +1630,7 @@ declare module 'I18n/en-US.json' {
'videoCallOverlayViewModeSpeakers': `Show active speakers only`;
'videoCallParticipantConnecting': `Connecting...`;
'videoCallParticipantLowerYourHand': `Lower your hand`;
'videoCallParticipantPressSpaceToUnmuteNotification': `Microphone temporarily on`;
'videoCallParticipantPressSpaceToUnmuteNotification': `Your microphone is on`;
'videoCallParticipantRaiseYourHand': `Raise your hand`;
'videoCallParticipantRaisedHandRaiseDuration': `Hand raised for {duration}`;
'videoCallParticipantRaisedSelfHandUp': `You have raised your hand up`;
Expand Down

0 comments on commit efc6e22

Please sign in to comment.