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

[Pull Request] A project level refactoring #278

Merged
merged 24 commits into from
Feb 25, 2024
Merged

[Pull Request] A project level refactoring #278

merged 24 commits into from
Feb 25, 2024

Conversation

Dynesshely
Copy link
Member

What does this PR do?

I'm happy to bring this PR, it's a huge change for the KitX Project.

First of all, what I will introduce is the new Dashboard <-> Loader communication method. We abandoned the previous Tcp Connection, but fully embraced WebSocket, and on this basis introduced ConnectionId as a runtime Plug-in distinction flag.

Secondly, we have improved the documentation and standards of KitX Network. Now, the development of Loaders suitable for other languages will be traceable.

In addition, for the first time we successfully called a plug-in method through KWC in KitX Network.

Here are the PR details:

Features:

Related issues

@Dynesshely Dynesshely added the enhancement New feature or request label Feb 25, 2024
@Dynesshely Dynesshely self-assigned this Feb 25, 2024
@Dynesshely
Copy link
Member Author

The origin title of this PR is: '[Pull Request] A project level refactoring'. GitHub Action is checking and format it ...

Dynesshely added a commit to Crequency/KitX-Loaders that referenced this pull request Feb 25, 2024
Dynesshely added a commit to Crequency/KitX-Plugins that referenced this pull request Feb 25, 2024
Dynesshely added a commit to Crequency/KitX-Dashboard that referenced this pull request Feb 25, 2024
Dynesshely added a commit to Crequency/KitX-Installer that referenced this pull request Feb 25, 2024
Dynesshely added a commit to Crequency/KitX-Mobile that referenced this pull request Feb 25, 2024
Copy link
Member

@Cronyet Cronyet left a comment

Choose a reason for hiding this comment

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

Epic update !

@Cronyet
Copy link
Member

Cronyet commented Feb 25, 2024

@Dynesshely You forget to update README.md.

@Dynesshely Dynesshely merged commit 7a58da4 into main Feb 25, 2024
6 checks passed
@github-actions github-actions bot added the merged Pull Request had been merged label Feb 25, 2024
Dynesshely added a commit to Crequency/KitX-SDK that referenced this pull request Feb 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request merged Pull Request had been merged
Projects
None yet
2 participants