Skip to content

[module][yaml] Moves sql modules to zone/npc/mob yamls - #11382

Draft
Frankie-hz wants to merge 1 commit into
LandSandBoat:basefrom
Frankie-hz:yamlmodules
Draft

[module][yaml] Moves sql modules to zone/npc/mob yamls#11382
Frankie-hz wants to merge 1 commit into
LandSandBoat:basefrom
Frankie-hz:yamlmodules

Conversation

@Frankie-hz

@Frankie-hz Frankie-hz commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

I affirm:

  • I understand that if I do not agree to the following points by completing the checkboxes my PR will be ignored.
  • I understand I should leave resolving conversations to the LandSandBoat team so that reviewers won't miss what was said.
  • I have read and understood the Contributing Guide and the Code of Conduct.
  • I have tested my code and the things my code has changed since the last commit in the PR and will test after any later commits.

What does this pull request do?

Changes the now outdated sql modules that do not work anymore into yaml.

Note: These also will also NOT work until #11283 is merged, which is waiting for some upstream yaml changes.

These were tested with the future PR change and they do work.

Steps to test these changes

Add the modules to the init, see them work eventually.

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

✨ Hello and thanks for the PR! ✨

🤖: This is a friendly automated reminder that the maintainers won't look at your PR until you've properly completed all of the checkboxes in the pre-filled template.

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