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

feat: custom npc shop items from lua table #1947

Merged
merged 4 commits into from
Dec 1, 2023

Conversation

dudantas
Copy link
Contributor

@dudantas dudantas commented Dec 1, 2023

This introduces a new feature that allows for the creation of NPC shop items from lua dynamic tables. It is now possible to associate specific items with keywords/storage and others methods, and when a player interacts with an NPC shop, the items related to those methods will be displayed in the shop's inventory.

Key changes included in this pull request:

  • Added support for associating items custom tables in the NPC shop system.
  • Implemented the logic to display items from hireling based on keywords when opening an NPC shop.

image
image
image
image
image

@luan
Copy link
Contributor

luan commented Dec 1, 2023

@dudantas dudantas force-pushed the dudantas/feat-npc-shop-based-on-keywords branch from e5af2a5 to 6bf229e Compare December 1, 2023 01:05
@dudantas dudantas changed the title feat: npc shop items based on various keywords feat: npc shop items based on keywords Dec 1, 2023
@dudantas dudantas changed the title feat: npc shop items based on keywords feat: custom npc shop items from lua table Dec 1, 2023
@dudantas
Copy link
Contributor Author

dudantas commented Dec 1, 2023

@dudantas maybe do https://github.com/opentibiabr/canary/blob/main/data-otservbr-global/npc/battlemart.lua also? since he sells so much stuff.

Good idea, I'll do it, thanks for remembering.

@dudantas dudantas force-pushed the dudantas/feat-npc-shop-based-on-keywords branch 2 times, most recently from dd0f466 to 012f273 Compare December 1, 2023 05:24
@dudantas dudantas force-pushed the dudantas/feat-npc-shop-based-on-keywords branch from 82381b9 to 8e297dc Compare December 1, 2023 05:29
Copy link

sonarcloud bot commented Dec 1, 2023

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 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@mattyx14
Copy link

mattyx14 commented Dec 1, 2023

gread .. goood

@dudantas dudantas merged commit 0e2d7fd into main Dec 1, 2023
38 checks passed
@dudantas dudantas deleted the dudantas/feat-npc-shop-based-on-keywords branch December 1, 2023 16:33
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.

6 participants