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.

Sign up for GitHub

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

Jump to bottom

Implement buttons for spending resources for actions #200

Merged
merged 13 commits into from
Jan 25, 2025
Merged

Implement buttons for spending resources for actions #200

merged 13 commits into from
Jan 25, 2025

Conversation

Azurelol
Copy link
Collaborator

@Azurelol Azurelol commented Jan 13, 2025 •
edited
Loading

  • Adds a data model for action items that require resources to be spent in using them, ActionCostDataModel
  • Adds a partial template for said model that can be used on item sheets such as Spell, Skill, Misc. Ability,
  • Integrates this action use through the resource pipeline
  • Provides revert for the resource loss.

image

@Azurelol Azurelol force-pushed the add/action_cost_buttons branch from e69b2f4 to 35c8bda Compare January 14, 2025 16:24
@Azurelol Azurelol changed the base branch from add/inline_weapon to dev January 14, 2025 16:30
@Azurelol Azurelol force-pushed the add/action_cost_buttons branch from 35c8bda to 301fdd1 Compare January 16, 2025 18:05
@Shourn Shourn self-requested a review January 19, 2025 07:55
Shourn
Shourn requested changes Jan 19, 2025
View reviewed changes
Copy link
Collaborator

@Shourn Shourn left a comment

Choose a reason for hiding this comment

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

Please have a look at the comments, if something is unclear please feel free to ask for clarification.
Please also resolve the merge conflicts with dev.

@Azurelol Azurelol force-pushed the add/action_cost_buttons branch 2 times, most recently from 8c81533 to 9e73445 Compare January 19, 2025 19:31
@Azurelol Azurelol force-pushed the add/action_cost_buttons branch from 54719c1 to 0edb540 Compare January 19, 2025 19:54
@Azurelol Azurelol requested a review from Shourn January 19, 2025 20:31
@Azurelol Azurelol marked this pull request as ready for review January 19, 2025 20:31
@Shourn Shourn merged commit be297e4 into dev Jan 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Shourn Shourn Awaiting requested review from Shourn

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Azurelol @Shourn