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

[Enhancement] reorganize reload functions and extract in an own class on cpp #669

Merged
merged 18 commits into from
Dec 8, 2022

Conversation

dudantas
Copy link
Contributor

@dudantas dudantas commented Dec 6, 2022

• Fixed some bugs in the reload, moved the reload function to its own lib, which is the beginning of the migration of all talkactions to this lib, this will allow us to:
Use the "/reload talkactions" command, thus returning to the old way of reloading only certain classes.

• Fixed crash related to reload monster and reload scripts
• Fixed some reloads not working correctly
• Created a new class for reload, facilitating the understanding and organization of functions.

@murilo09 murilo09 linked an issue Dec 8, 2022 that may be closed by this pull request
5 tasks
@dudantas dudantas changed the title Reorganize reload functions and extract in a own class [Enhancement] reorganize reload functions and extract in an own class on cpp Dec 8, 2022
@sonarcloud
Copy link

sonarcloud bot commented Dec 8, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@dudantas dudantas merged commit fe2a920 into main Dec 8, 2022
@dudantas dudantas deleted the reorganize-reload-functions branch December 8, 2022 11:38
luan pushed a commit that referenced this pull request Jul 11, 2023
… on cpp (#669)

• Fixed some bugs in the reload, moved the reload function to its own lib, which is the beginning of the migration of all talkactions to this lib, this will allow us to:
Use the "/reload talkactions" command, thus returning to the old way of reloading only certain classes.

• Fixed crash related to "/reload monster" and "/reload scripts"
• Fixed some reloads not working correctly
• Fixed "/reload npc"
• Created a new class for reload, facilitating the understanding and organization of functions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

/Reload npc not reloading npcs
3 participants