Skip to content

[sql, core] Adds content tags to mobgroups#8398

Merged
Xaver-DaRed merged 1 commit into
LandSandBoat:basefrom
Frankie-hz:mobgrouptags
Jan 21, 2026
Merged

[sql, core] Adds content tags to mobgroups#8398
Xaver-DaRed merged 1 commit into
LandSandBoat:basefrom
Frankie-hz:mobgrouptags

Conversation

@Frankie-hz
Copy link
Copy Markdown
Contributor

@Frankie-hz Frankie-hz commented Oct 11, 2025

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?

Adds content tags to mobgroups in order to turn off certain out of era mob groups and NMs.

See: #8093 for things it can simplify.

Steps to test these changes

Run: UPDATE mob_groups SET content_tag = 'WOTG' WHERE NAME = "Orcish_Grappler";
UPDATE mob_groups SET content_tag = 'WOTG' WHERE NAME = "Orcish_Mesmerizer";
!gotoid 17350669
See only fodders in the area. This was tested with and without spawn slots

image

@WinterSolstice8 WinterSolstice8 added the hold On hold, pending further action/info label Oct 13, 2025
@Xaver-DaRed
Copy link
Copy Markdown
Contributor

Pending PR #9111 for this to be revisisted

@Frankie-hz Frankie-hz closed this Jan 15, 2026
@Frankie-hz Frankie-hz reopened this Jan 15, 2026
@WinterSolstice8 WinterSolstice8 removed the hold On hold, pending further action/info label Jan 15, 2026
@Frankie-hz Frankie-hz force-pushed the mobgrouptags branch 2 times, most recently from d7a2884 to 0e7efb1 Compare January 21, 2026 02:33
@Xaver-DaRed Xaver-DaRed merged commit 5f8c738 into LandSandBoat:base Jan 21, 2026
10 checks passed
@Frankie-hz Frankie-hz deleted the mobgrouptags branch January 26, 2026 05:10
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.

3 participants