Skip to content
This repository has been archived by the owner on May 20, 2023. It is now read-only.

[Enhancement] Protocol update to 12.91 #732

Merged
merged 17 commits into from
Sep 15, 2022

Conversation

guispiller
Copy link
Contributor

@guispiller guispiller commented Jul 26, 2022

Description

Update datapack items and npcs to protocol 12.91;
Changed some monster's loot names to ID to avoid conflict;
Inserted some Marapur monsters by @carlospess0a and @JohneVictor
Inserted transformto logics of items.

This PR works directly with Canary opentibiabr/canary#443.

If you are interested in testing 12.90 items with their special skills, you could use this PR with opentibiabr/canary#366 and remove the comment from attributes on items.xml.

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

@guispiller guispiller changed the title Items 1287 Protocol Update from items to 12.90 Jul 26, 2022
@guispiller guispiller changed the title Protocol Update from items to 12.90 [Enhancement] Protocol Update from items to 12.90 Jul 26, 2022
@carlospess0a
Copy link

carlospess0a commented Aug 28, 2022

What do you think to use tables as parameters to avoid mass updates in npc's?

npcType.onBuyItem = function(npc, player, itemId, subType, amount, ignore, inBackpacks, totalCost)

to

npcType.onBuyItem = function(npcParameters)

npcParameters.npc
npcParameters.player
etc...

@majestyotbr majestyotbr changed the title [Enhancement] Protocol Update from items to 12.90 [Enhancement] Protocol update to 12.91 Sep 9, 2022
@dudantas dudantas merged commit 5dae66e into opentibiabr:main Sep 15, 2022
@guispiller guispiller deleted the items-1287 branch September 19, 2022 15:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] buy shoping in bags
5 participants