Skip to content

Conversation

@beats-dh
Copy link
Collaborator

Description

Introduces a new Free Quests system with C++ backend integration, replacing the previous Lua-based implementation. Adds game_freequests.cpp/hpp, updates player login logic to apply free quests, and exposes Game.addFreeQuestData to Lua. Refactors quest data loading and validation, improves date/time formatting functions, and updates .gitignore and build configuration. Also fixes time formatting in database backup and tools.cpp to use chrono time points.

@beats-dh beats-dh force-pushed the beats-fix-and-freeQuestCPP branch from 0a90c20 to 9e02501 Compare October 19, 2025 02:45
@sonarqubecloud
Copy link

@dudantas
Copy link
Member

This change isn't beneficial because it moves a component that was previously bound to the global pack to something loaded across the entire server, even though it's only needed by the global pack. This increases compilation time, distributes unused code across all scenarios, and makes maintenance and review more difficult, without any apparent benefit.

@lgrossi lgrossi force-pushed the beats-fix-and-freeQuestCPP branch 3 times, most recently from 69661ed to f614712 Compare November 26, 2025 07:35
Introduces a new Free Quests system with C++ backend integration, replacing the previous Lua-based implementation. Adds game_freequests.cpp/hpp, updates player login logic to apply free quests, and exposes Game.addFreeQuestData to Lua. Refactors quest data loading and validation, improves date/time formatting functions, and updates .gitignore and build configuration. Also fixes time formatting in database backup and tools.cpp to use chrono time points.
@lgrossi lgrossi force-pushed the beats-fix-and-freeQuestCPP branch from f614712 to 032a6db Compare November 26, 2025 07:44
@sonarqubecloud
Copy link

@opentibiabr opentibiabr deleted a comment from github-actions bot Nov 26, 2025
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