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

9.6.3 - Versatile Vials missing item bonus(es) to attack #16201

Closed
aremedis opened this issue Aug 23, 2024 · 5 comments
Closed

9.6.3 - Versatile Vials missing item bonus(es) to attack #16201

aremedis opened this issue Aug 23, 2024 · 5 comments
Labels
consolidated A consolidated issue made from multiple other issues feature request

Comments

@aremedis
Copy link

At levels 4,12, 18 in addition to damage die number increasing there should also be a +1, +2, +3 item bonus to attack rolls.

Probably a better way to do this, but this is what I've come up with:

{ "key": "FlatModifier", "value": { "brackets": [ { "end": 3, "value": 0 }, { "start": 4, "end": 11, "value": 1 }, { "start": 12, "end": 17, "value": 2 }, { "start": 18, "value": 3 } ], "field": "actor|level" }, "selector": "attack", "type": "item" },

@pedrogrullada
Copy link
Collaborator

The bonuses exist and are in the Versatile Vials class feature for now. The rule elements can't be in the item itself because there is no way for the vials to reference back to who crafted them, and thus giving an accurate bonus (for example, if differently leveled allies are using them).

Once the system has come out with its implementation of Crafting, these features will be looked at again.

@nikolaj-a
Copy link
Collaborator

Aren't the stats based on the level of the versatile vial? They follow the normal lesser, moderate, greater, and major tiers.

@pedrogrullada
Copy link
Collaborator

pedrogrullada commented Aug 23, 2024

Aren't the stats based on the level of the versatile vial? They follow the normal lesser, moderate, greater, and major tiers.

I was told to implement them as a single item. Presumably there is a plan to handle it once we have proper system implementation.

@ammalagonc
Copy link
Collaborator

I'll reference this to the PC2 Alchemist changes tracking and consolidate.
#15647

@ammalagonc ammalagonc added feature request consolidated A consolidated issue made from multiple other issues labels Aug 23, 2024
@ammalagonc
Copy link
Collaborator

Closing this issue - tracking via #15647

@ammalagonc ammalagonc closed this as not planned Won't fix, can't repro, duplicate, stale Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
consolidated A consolidated issue made from multiple other issues feature request
Projects
None yet
Development

No branches or pull requests

4 participants