Skip to content

Add role and delivery quests#7

Merged
QuackieMackie merged 1 commit intoQuackieMackie:masterfrom
KarboniteKream:feat/role-quests
Feb 8, 2026
Merged

Add role and delivery quests#7
QuackieMackie merged 1 commit intoQuackieMackie:masterfrom
KarboniteKream:feat/role-quests

Conversation

@KarboniteKream
Copy link
Copy Markdown
Contributor

This PR adds the Roles and Deliveries categories under the Class & Jobs tab.

if (!isInitialized)
{
InitializeSubDirDropDown();
InitializeDropDown();
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Applied your changes from #7

(GetCategoryIdByName("Disciple of War Job Quests") ?? 0, "Jobs", 0, null),
(GetCategoryIdByName("Disciple of Magic Job Quests") ?? 0, "Jobs", 0, null),

(GetCategoryIdByName("Role Quests") ?? 0, "Roles", 0, null),
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This adds a bunch of sub-categories per expansion (same as the JSON filename). I tried making a Tank Role Quests category and merging the quests under it, but they're not sorted nicely (expansions are mixed). So I think this might be a good compromise.

@QuackieMackie QuackieMackie self-requested a review February 7, 2026 23:18
@QuackieMackie QuackieMackie merged commit 1dff426 into QuackieMackie:master Feb 8, 2026
1 check passed
@KarboniteKream KarboniteKream deleted the feat/role-quests branch February 9, 2026 04:01
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.

2 participants