Skip to content
This repository has been archived by the owner on May 14, 2024. It is now read-only.

Update readme.md with correct ports #2199

Merged
merged 1 commit into from
Mar 26, 2023

Conversation

bitbucket90
Copy link
Contributor

@bitbucket90 bitbucket90 commented Mar 21, 2023

To correctly query the server from funcoms master we need to allow port 7778. Once done this will show the correct age & ping.

Description

Checklist for all submissions

  • [X ] Have you followed the guidelines in our Contributing document?
  • [ X] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • [ X] 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?: In line testing, just updated readme
  • 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. [X ] 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 (e.g /game_eggs) 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

To correctly query the server from funcoms master we need to allow port 7778. Once done this will show the correct age & ping.
@parkervcp parkervcp merged commit 8a02f32 into pelican-eggs:master Mar 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants