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

[#1401] Item Choice Advancement #2015

Merged
merged 7 commits into from
Jan 25, 2023

Conversation

arbron
Copy link
Collaborator

@arbron arbron commented Jan 2, 2023

Introduces the item choice advancement for choosing one or more options when you reach a level, usable for things like Fighting Style, Metamagic, or Signature Spells.

The player can select an option from a predefined list like Fighting Style:

Item.Choice.-.Basic.webm

Or they can drop items onto the advancement such as with Magical Secrets:

Item.Choice.-.Drops.webm

If a choice is available over multiple levels like Metamagic, the previously made choices will be shown:

Item.Choice.-.Previous.Choices.webm

Configuration is handled through an interface similar to Grant Items, but with the level list allowing to specify how many choices are available at which levels:

Item Choice configuration interface

This also adds the ability to configure spells that are added through Item Grant or Item Choice advancements, setting ability, preparation mode, and limited uses:

Item Grant - Spell Config

Resolves #1401

To Do

  • Add item choice advancements to SRD classes & subclasses

@arbron arbron marked this pull request as draft January 2, 2023 19:26
@arbron arbron force-pushed the advancement/item-choice-rework branch 2 times, most recently from b91b30c to b2c12ab Compare January 3, 2023 04:56
@arbron arbron changed the title [#1401] Expand ItemGrant to support choices [#1401] Item Choice Advancement Jan 3, 2023
@arbron arbron self-assigned this Jan 3, 2023
@arbron arbron requested a review from Fyorl January 3, 2023 04:57
@arbron arbron marked this pull request as ready for review January 3, 2023 04:57
@arbron arbron added this to the D&D5E 2.1.0 milestone Jan 3, 2023
@arbron arbron force-pushed the advancement/item-choice-rework branch from b2c12ab to 104a543 Compare January 3, 2023 05:00
@arbron arbron modified the milestones: D&D5E 2.1.0, D&D5E 2.2.0 Jan 3, 2023
Copy link
Contributor

@Fyorl Fyorl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this will be a very powerful advancement, nice work.

less/advancement.less Outdated Show resolved Hide resolved
module/applications/advancement/item-choice-config.mjs Outdated Show resolved Hide resolved
module/applications/advancement/item-choice-flow.mjs Outdated Show resolved Hide resolved
module/applications/advancement/item-choice-flow.mjs Outdated Show resolved Hide resolved
module/applications/advancement/item-choice-flow.mjs Outdated Show resolved Hide resolved
module/documents/advancement/item-choice.mjs Outdated Show resolved Hide resolved
module/documents/advancement/item-grant.mjs Show resolved Hide resolved
templates/advancement/item-choice-config.hbs Show resolved Hide resolved
templates/advancement/item-choice-flow.hbs Outdated Show resolved Hide resolved
module/documents/advancement/item-choice.mjs Outdated Show resolved Hide resolved
@arbron arbron force-pushed the advancement/item-choice-rework branch from 8a9b4f9 to eb0de39 Compare January 15, 2023 22:33
@arbron arbron linked an issue Jan 22, 2023 that may be closed by this pull request
@arbron arbron changed the base branch from 2.1.x to 2.2.x January 25, 2023 17:03
@arbron arbron merged commit 15fa75b into foundryvtt:2.2.x Jan 25, 2023
@arbron arbron deleted the advancement/item-choice-rework branch January 25, 2023 17:03
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.

Advancement Type: Item Choice
2 participants