Skip to content

Update CloudLink#2458

Open
MikeDev101 wants to merge 11 commits intoTurboWarp:masterfrom
MikeDev101:patch-1
Open

Update CloudLink#2458
MikeDev101 wants to merge 11 commits intoTurboWarp:masterfrom
MikeDev101:patch-1

Conversation

@MikeDev101
Copy link
Copy Markdown
Contributor

This is my (hopefully) final PR relating to Classic CloudLink.

This is a slightly altered variant of "CloudLink Improved", created by @Mistium.
https://github.com/Mistium/extensions.mistium/blob/main/files%2FCloudlink4_Improved.js

This addresses miscellaneous bugs encountered since the last major PR and "shuts up" the logs.

I do not take credit for the many wonderful contributions provided by Mist.

This is my (hopefully) final PR relating to Classic CloudLink.

This is a slightly altered variant of CloudLink Improved, created by @Mistium (https://github.com/Mistium/extensions.mistium/blob/main/files%2FCloudlink4_Improved.js).

This addresses various bugs encountered since the last major PR and enhances QoL.
@github-actions github-actions bot added the pr: change existing extension Pull requests that change an existing extension label Apr 10, 2026
@MikeDev101

This comment was marked as outdated.

@@ -1,11 +1,20 @@
// Name: CloudLink V4
// Name: Cloudlink
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

the version number seems somewhat important? since there are others

@Mistium
Copy link
Copy Markdown

Mistium commented Apr 10, 2026

@MikeDev101 ive made some upstream changes for cl4 improved, commit here: Mistium/extensions.mistium@3d88928

i decided to clean up some of the issues raised, but some stuff like the local server list are things that i see as less than necessary to implement but i understand the requirement so ill leave them to mike to copy over from the original version of the extension

Merged changes from

Mistium/extensions.mistium@3d88928

and restored the static serverlist property.
@MikeDev101

This comment was marked as outdated.

@MikeDev101
Copy link
Copy Markdown
Contributor Author

!format

@CubesterYT
Copy link
Copy Markdown
Member

@MikeDev101 you still have some reviews that you need to address

@MikeDev101
Copy link
Copy Markdown
Contributor Author

@MikeDev101 you still have some reviews that you need to address

just finished addressing them 🦆

@MikeDev101
Copy link
Copy Markdown
Contributor Author

I could be wrong, but as of b0d4615 it should be safe now.

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

Labels

pr: change existing extension Pull requests that change an existing extension

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants