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

Implement Folia Support #2682

Draft
wants to merge 21 commits into
base: v3
Choose a base branch
from
Draft

Implement Folia Support #2682

wants to merge 21 commits into from

Conversation

SirYwell
Copy link
Member

Rules (read carefully)

This PR will be around for quite some time most likely, so we need a few rules:

  • General
    • Neither Folia nor this PR are production ready. Expect your server to crash and to hang. Expect to loose (world) data.
    • Only use this software when you're able to debug problems. There are enough people who can run commands until something breaks. There are not enough people who can analyze and resolve such problems.
    • If you use modifications of Folia or FAWE, expect even more things to break. We won't look into your problems in such
  • Reporting Bugs
    • Make sure that you don't report bugs that were reported already.
    • Make sure you can reproduce the bug on the latest Folia build with no other plugins installed.
    • There is no benefit in reporting bugs we cannot reproduce at this stage.
    • This PR targets the next major FAWE version. Before reporting bugs, make sure those bugs are not present on the main branch or the v3 branch.
  • Contributing
    • If you want to help, make sure you understand what's going on.
    • Keep your changes small. I won't look through thousand lines of code.
    • We do not try to re-enable features that just aren't supported by Folia at the moment.
    • Don't try to implement something just to have it implemented. Only implement things you can reason about.
    • Open a PR targeting this branch. Expect your PR to be reviewed carefully. Expect your PR to be rejected.
    • Discussing changes with us before will increase the likelihood of changes useful for us.

Known Bugs and Limitations

  • //regen does not work as it requires world loading
  • Copying entities does not work
  • Copying blocks with block entities does not work well

Submitter Checklist

Edit tasklist title
Beta Give feedback Tasklist Submitter Checklist, more options

Delete tasklist

Delete tasklist block?
Are you sure? All relationships in this tasklist will be removed.
  1. Make sure you are opening from a topic branch (/feature/fix/docs/ branch (right side)) and not your main branch.
    Options
  2. Ensure that the pull request title represents the desired changelog entry.
    Options
  3. New public fields and methods are annotated with @since TODO.
    Options
  4. I read and followed the contribution guidelines.
    Options

@github-actions github-actions bot added the Feature This PR adds a new feature label Apr 11, 2024
@Bufferless
Copy link

Issues that I have found when using this version with the latest Folia 1.20.4 without any plugins besides FAWE:

  • //up 1 will send an error and not work, it will send "Editing on unsupported versions is disabled"
  • Making big changes with FAWE or loading a big schematic will cause FAWE to no longer respond to any worldedit / fawe commands.
  • After setting the pos2 with the //wand, it will no longer respond if you try right-clicking a different area making you have to use the command //pos2.
  • You will have to relog in order to see changes made with huge edits.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature This PR adds a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants