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

Update SetupMinecraft.sh #143

Merged
merged 2 commits into from
Dec 1, 2024
Merged

Conversation

M-Taylor40
Copy link
Contributor

Updating DownloadURL with updated download links.

Updating DownloadURL with updated download links.
@M-Taylor40
Copy link
Contributor Author

Downloads are no longer using azureedge.net URLs.

@M-Taylor40 M-Taylor40 closed this Sep 23, 2024
@M-Taylor40 M-Taylor40 reopened this Sep 23, 2024
@rlboyd
Copy link

rlboyd commented Sep 30, 2024

Any chance of getting this merged?

@skoelle
Copy link

skoelle commented Oct 7, 2024

@M-Taylor40 start.sh needs the change too

@M-Taylor40
Copy link
Contributor Author

M-Taylor40 commented Oct 7, 2024

Updated URLs within start.sh script as highlighted by skoelle.

@M-Taylor40
Copy link
Contributor Author

@TheRemote Please can you review and merge the changes to the new URL?

@TKlerx
Copy link

TKlerx commented Oct 26, 2024

Would be great if this PR could be merged.

@zbfgsp3k
Copy link

Albeit 1.21.43.01 being ahead of the current version, please get the PR merged. Thank you

@0x233233
Copy link

0x233233 commented Nov 4, 2024

To temporary solve this issues, i created a fork, patched it with the new url like M-Taylor40 has done and created a script to change the minecraft start, stop, etc. scripts to my repo. So other users have an easy solution.

https://github.com/0x233233/MinecraftBedrockServer

@TheRemote TheRemote merged commit 347e961 into TheRemote:master Dec 1, 2024
@TheRemote
Copy link
Owner

Accepted, thank you!

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.

7 participants