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

Innate spellcasting missing required components #475

Closed
Rarst opened this issue Jun 30, 2022 · 1 comment · Fixed by #479
Closed

Innate spellcasting missing required components #475

Rarst opened this issue Jun 30, 2022 · 1 comment · Fixed by #479

Comments

@Rarst
Copy link
Contributor

Rarst commented Jun 30, 2022

The most common components note for innate spellcasting is requiring no material components and the data for it is an empty array "components_required": [].

I think this should be ["V","S"] since nothing is said about verbal and somatic, thus they are still required?

For comparison requiring only verbal components is captured in data as components_required": ["V"].

Relevant Jeremy Crawford tweet:

- innate spellcasting doesn't require material components. Verbal and somatic?
- Casting a spell requires all its components unless a trait or feature says otherwise.

https://twitter.com/JeremyECrawford/status/606207225334530048

@bagelbits
Copy link
Collaborator

I think you're right. Good catch! Feel free to cut a PR if you want to fix this.

Rarst added a commit to Rarst/5e-database that referenced this issue Jul 1, 2022
Only material is specifically excluded for these, rest of components apply as normal.

Fixes 5e-bits#475
bagelbits added a commit that referenced this issue Jul 6, 2022
Only material is specifically excluded for these, rest of components apply as normal.

Fixes #475

Co-authored-by: Chris Ward <cdurianward@gmail.com>
bagelbits added a commit that referenced this issue Jul 6, 2022
Only material is specifically excluded for these, rest of components apply as normal.

Fixes #475

Co-authored-by: Chris Ward <cdurianward@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants