Skip to content

Fix the approval question and the session state, keep the viewer live - #1

Merged
jbleyel merged 1 commit into
mainfrom
fix-approval-and-viewer
Oct 2, 2026
Merged

jbleyel merged 1 commit into
mainfrom
fix-approval-and-viewer

Conversation

@WXbet

@WXbet WXbet commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Three issues found while taking screenshots of a session on a receiver:

  • Approval question after a rename: a participant renamed while the approval question was open was still asked for with the old name ("User 2") until the question timed out and counted as not approved. The question is now withdrawn and asked again with the new name.
  • Session state after everybody left: the screen and the indicator kept showing "access approved" after the approval was dropped because nobody approved was connected anymore. They are updated now when the approval changes.
  • Viewer: UP on a terminal without history held its output back without showing that, so the view seemed to stop until it was reopened. It only pauses now when there is something to scroll back to.

A participant renamed while the approval question was open was still
asked for with the old name until the question timed out and counted as
not approved. The question is now withdrawn and asked again with the new
name.

The screen and the indicator kept showing the access as approved after
everybody approved had left, they are updated now when the approval is
dropped.

UP in the viewer on a terminal without history held its output back
without showing it, so the view seemed to stop. It only pauses now when
there is something to scroll back to.
@sonarqubecloud

sonarqubecloud Bot commented Oct 2, 2026

Copy link
Copy Markdown

@jbleyel
jbleyel merged commit 1f39c3c into main Oct 2, 2026
5 checks passed
@WXbet
WXbet deleted the fix-approval-and-viewer branch October 2, 2026 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants