Skip to content

Clean up TypeScript types related to tsh daemon#40017

Merged
ravicious merged 6 commits intomasterfrom
r7s/move-tshd-client
Apr 2, 2024
Merged

Clean up TypeScript types related to tsh daemon#40017
ravicious merged 6 commits intomasterfrom
r7s/move-tshd-client

Conversation

@ravicious
Copy link
Copy Markdown
Member

  • VnetServiceClient is now VnetClient.
  • TshdClient, VnetClient and everything else needed by the outside world is exported from web/packages/teleterm/src/services/tshd/index.ts.
  • tshd/types.ts is deprecated in favor of importing directly from gen-proto-ts and moving types to files centered around concepts and data structures, rather than having a dedicated types.ts file.

I'll have to manually backport this as v15 doesn't know and won't know about VNet.

@ravicious ravicious added no-changelog Indicates that a PR does not require a changelog entry backport/branch/v15 labels Mar 29, 2024
@ravicious ravicious requested a review from gzdunek March 29, 2024 13:58
@github-actions github-actions Bot requested review from avatus and rudream March 29, 2024 13:58
@ravicious ravicious removed the request for review from rudream March 29, 2024 14:03
Copy link
Copy Markdown
Contributor

@gzdunek gzdunek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So clean now 🙌

Comment thread web/packages/teleterm/src/services/tshd/types.ts Outdated
Comment thread web/packages/teleterm/src/services/tshd/createClient.ts Outdated
@ravicious ravicious enabled auto-merge April 2, 2024 09:41
@ravicious ravicious added this pull request to the merge queue Apr 2, 2024
Merged via the queue into master with commit 4c0d5d0 Apr 2, 2024
@ravicious ravicious deleted the r7s/move-tshd-client branch April 2, 2024 10:02
@public-teleport-github-review-bot
Copy link
Copy Markdown

@ravicious See the table below for backport results.

Branch Result
branch/v15 Failed

@public-teleport-github-review-bot
Copy link
Copy Markdown

@ravicious See the table below for backport results.

Branch Result
branch/v15 Failed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog Indicates that a PR does not require a changelog entry size/sm ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants