Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Support for toolbox 2.6.0.38311#8

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
fioan89 merged 20 commits intomainfromsupport-for-toolbox-2_6_0_38311
Feb 27, 2025

Conversation

fioan89
Copy link
Collaborator

  • fixes login screen and glitches related jumping to the workspaces page once the connection to Coder deployment is estabilished.
  • support for opening URLs in browser
  • fixes user agent reporting
  • fixes connection status rendering
  • a couple of functions on the existing models and views transformed into class properties
  • while other functions are now suspend functions
  • upgrade kotlin dependencies because Toolbox 2.6.0.38311 runs with Kotlin stdlib 2.1.0
    therefore I've upgraded the Kotlin compiler to match the runtime.
  • similarly coroutines, serialization and KSP support had to be upgraded.

- will also reflect on the jar/zip name
- copied from coder gateway plugin
- the API for browsing url(s) is no longer available- this patch implements a basic URL opener that  relies on native tools.
- if successfully signed in we go back to the  main page which shows the environments/workspaces- similar approach if user hits Cancel during sign in- `EnvironmentUiPageManager` is an undocumented Toolbox  component that has more flexibility to navigate between pages
- the connection status was unreadable due to the background  and foreground colors used. The latest Toolbox version  comes with standard color palettes that change based  on the theme and environment state.- plus some small extra refactorings that reduce the number  of constructor parameters and rely on the service locator
- url was not displayed in the env page until the rest  client was connected- but we don't need to wait for the rest client to connect,  the url is already available from settings
- Toolbox does not provide a plugin manager interface with metadata about current plugin.  So we had to implement one for ourselves in order to have simple things like plugin version.- the implementation relies on the extension.json which is deployed in the root of the  jar (i.e. the plugin jar)- the rest of the json fields are ignored for now
- Toolbox 2.6.0.38311 comes with new plugin API dependencies
- Toolbox 2.6.0.38311 runs with Kotlin stdlib 2.1.0  therefore I've upgraded the Kotlin compiler to match the runtime.- similarly coroutines, serialization and KSP support had to be upgraded.
- due to new API for remote environment model
- due to new API for SSH environment view
- a couple of functions on the existing models and views  transformed into class properties- while other functions are now suspend functions
- the plugin was loading so no classloading issues related to logging classes, but it was not logging  anything from the plugin.- the tinylog dependency was apparently ignored. Until we receive a definitive answer from JetBrains  we have two options: either propagate the service locator and the logging service to every component  (even non UI models/views) or wrap the toolbox logging service in a slf4j implementation with minimal  invasive changes.- the latter approach is possible because no component is initialized until CoderToolboxExtension is called.  That means we have the chance to initialize our custom logger factory around the toolbox logger service.
@fioan89fioan89 merged commitf2dc464 intomainFeb 27, 2025
@fioan89fioan89 deleted the support-for-toolbox-2_6_0_38311 branchFebruary 27, 2025 20:08
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@matifalimatifalimatifali approved these changes

@bcpeinhardtbcpeinhardtAwaiting requested review from bcpeinhardt

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@fioan89@matifali

[8]ページ先頭

©2009-2025 Movatter.jp