Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix#8912: clear SSH service messages not working #8918

Merged
merged 2 commits into from
Oct 6, 2023

Conversation

Clem-Fern
Copy link
Contributor

@Clem-Fern Clem-Fern commented Sep 2, 2023

Hey @Eugeny, I hope you are doing well.

This PR aims to fix the issue #8912. By adding a global connectable tab option for clearServiceMessagesOnConnect in #8726 ef6b8a4, I placed the fronted clear call before the session finishes initializing. It results in only cleaning latest terminal session log on reconnection and not service message on connection, which, I think, is not wanted from what I see in #7841.

Feel free to ask if any change are needed.

Resolve #8912

@Clem-Fern Clem-Fern marked this pull request as draft September 2, 2023 18:13
@Clem-Fern Clem-Fern marked this pull request as ready for review September 2, 2023 18:40
@Eugeny
Copy link
Owner

Eugeny commented Oct 6, 2023

LGTM 🚀

@Eugeny Eugeny merged commit adeb603 into Eugeny:master Oct 6, 2023
10 checks passed
@Issues-translate-bot
Copy link
Collaborator

The translator bot has detected that this issue body's language is not English, and has translated it automatically.


LGTM 🚀

@Clem-Fern Clem-Fern deleted the fix#8912 branch October 6, 2023 11:45
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.

1.0.200 - clear SSH service messages by default (not working)
3 participants