Skip to content

[7DTD] Add Crossplay parameter and variables needed for upcoming 2.0 version - #179

Open
ScattyNatalie wants to merge 6 commits into
pelican-eggs:mainfrom
ScattyNatalie:7dtd-2.0-update
Open

[7DTD] Add Crossplay parameter and variables needed for upcoming 2.0 version#179
ScattyNatalie wants to merge 6 commits into
pelican-eggs:mainfrom
ScattyNatalie:7dtd-2.0-update

Conversation

@ScattyNatalie

Copy link
Copy Markdown

Description

  • Added ServerAllowCrossplay parameter and Variable to the Egg
  • Added EACEnabled parameter and Variable to the Egg
  • Added IgnoreEOSSanctions parameter and Variable to the Egg
  • Added instructions for what variables to set in order to enable crossplay for 7DTD dedicated server in the readme
  • Added a update URL

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

@ScattyNatalie

Copy link
Copy Markdown
Author

Also as with my other pr, I don't have a ptero panel setup in order to export a ptero egg.

Comment thread 7_days_to_die/egg-7-days-to-die.json Outdated
"name": "7 Days To Die",
"author": "kristoffer.norman@bahnhof.se",
"uuid": "b7a88721-15c1-4081-beed-e3add926a372",
"uuid": "b1545a0f-f8ba-41dc-9afc-3020696c08be",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
"uuid": "b1545a0f-f8ba-41dc-9afc-3020696c08be",
"uuid": "b7a88721-15c1-4081-beed-e3add926a372",

UUID don't change between Pterodactyl & Pelican

Comment on lines +70 to +74
"rules": [
"required",
"string",
"in:true,false"
]

@rmartinoscar rmartinoscar Jun 20, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
"rules": [
"required",
"string",
"in:true,false"
]
"rules": [
"required",
"boolean"
]

https://laravel.com/docs/12.x/validation#rule-boolean

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

boolean for us is 0 and 1 not true and false

Comment on lines +56 to +60
"rules": [
"required",
"string",
"in:true,false"
]

@rmartinoscar rmartinoscar Jun 20, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
"rules": [
"required",
"string",
"in:true,false"
]
"rules": [
"required",
"boolean"
]

https://laravel.com/docs/12.x/validation#rule-boolean

Comment on lines +98 to +102
"rules": [
"required",
"string",
"in:true,false"
]

@rmartinoscar rmartinoscar Jun 20, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
"rules": [
"required",
"string",
"in:true,false"
]
"rules": [
"required",
"boolean"
]

https://laravel.com/docs/12.x/validation#rule-boolean

@ScattyNatalie

Copy link
Copy Markdown
Author

Alright ill work on those right now. Also no idea why it changed the uuid

@ScattyNatalie

Copy link
Copy Markdown
Author

If there's any other changes you would like to me to just let me know.

@ScattyNatalie

ScattyNatalie commented Jun 20, 2025

Copy link
Copy Markdown
Author

Actually when I put true or false it says this:
Ignore EOS Sanctions*
{{IGNORE_EOS_SANCTIONS}}
false
The Ignore EOS Sanctions field must be true or false.

And this is with the Boolean rule. 1 or 0 works but not true or false

@gOOvER

gOOvER commented Jun 20, 2025

Copy link
Copy Markdown
Contributor

Alright ill work on those right now. Also no idea why it changed the uuid

you dont edited the original egg and created a new one

@gOOvER

gOOvER commented Jun 20, 2025

Copy link
Copy Markdown
Contributor

Actually when I put true or false it says this: Ignore EOS Sanctions* {{IGNORE_EOS_SANCTIONS}} false The Ignore EOS Sanctions field must be true or false.

And this is with the Boolean rule. 1 or 0 works but not true or false

in:true,false

you have an example there. Scroll down :)

@ScattyNatalie

Copy link
Copy Markdown
Author

There adding in the in:true,false seemed to fix it

@redthirten redthirten changed the title Add Crossplay parameter and variables needed for upcoming 2.0 version [7DTD] Add Crossplay parameter and variables needed for upcoming 2.0 version Jun 20, 2025
@okseb

okseb commented Jul 2, 2025

Copy link
Copy Markdown

Is this planned to be updated for pterodactyl users as well?

@QuintenQVD0

Copy link
Copy Markdown
Contributor

Is this planned to be updated for pterodactyl users as well?

Yes it should be.

@redthirten

Copy link
Copy Markdown
Contributor

@CuteNatalie If you want, you can try using this tool I recently made to convert your Pelican Egg to a Ptero Egg for inclusion in your PR: https://redthirten.github.io/scramble-egg-converter/

If not, we can also convert it after the merge as well.

@okseb

okseb commented Jul 6, 2025

Copy link
Copy Markdown

Can I try this out now or is it not ready yet?

@ScattyNatalie

Copy link
Copy Markdown
Author

Uploaded the ptero version

@ScattyNatalie

Copy link
Copy Markdown
Author

Can I try this out now or is it not ready yet?

Also yes it should be good to try out

@okseb

okseb commented Jul 6, 2025

Copy link
Copy Markdown

Before you responded I actually went ahead to try it anyway and converted it with the posted tool to Ptero, but when I try to create a server I get invalid data on the variables that need to be true or false. Apparently they aren't set to one of those 2 values when they are, maybe I can be advised what I'm doing wrong.

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.

6 participants