Skip to content

Valheim fix#438

Open
choren5150 wants to merge 2 commits intopterodactyl:mainfrom
choren5150:valheim-fix
Open

Valheim fix#438
choren5150 wants to merge 2 commits intopterodactyl:mainfrom
choren5150:valheim-fix

Conversation

@choren5150
Copy link

Description

Checklist for all submissions

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you tested and reviewed your changes with confidence that everything works?
  • Did you branch your changes and PR from that branch and not from your master branch?
    • If not, why?:
  • You verify that the start command applied does not use a shell script
    • If some script is needed then it is part of a current yolk or a PR to add one
  • The egg was exported from the panel

New egg Submissions

  1. Does your submission pass tests (server is connectable)?
  2. Does your egg use a custom docker image?
    • Have you tried to use a generic image?
    • Did you PR the necessary changes to make it work?
  3. Have you added the egg to the main README.md and any other README files in subdirectories of the egg according to the alphabetical order?
  4. Have you added a unique README.md for the egg you are adding according to the alphabetical order?
  5. You verify that the start command applied does not use a shell script
    • If some script is needed then it is part of a current yolk or a PR to add one
  6. The egg was exported from the panel

@choren5150
Copy link
Author

This change updates the startup command to use the latest variables with the latest release. It also addresses the need to currently use a longer named file for a temporary fix by the Valheim+ devs until the core Valheim development team fixes the localization issues. I did not update the base variable name of UnixServer.zip to UnixServerRenamed.zip that currently exists as part of Valheim+ 0.9.17.1 as the Valheim+ devs have asserted they will revert to the old name once they no longer have to leverage the current workaround.

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.

1 participant