reznok / GMCAbilitySystem

An Unreal Engine plugin that adds an ability system to the General Movement Component plugin
MIT License
31 stars 14 forks source link

Refactor StartingAbilityMap to an Array instead of a single variable #23

Closed LeoFabre closed 6 months ago

LeoFabre commented 7 months ago

This would be useful to have to enhance modularity.

For example, a project could have a DefaultAbilityMap and a ShootingAbilityMap, and users could choose to have only the first or have both added to their character, depending on the need to be able to shoot or just roam around unarmed.

We can also make the AbilityMap DataAsset to also hold tags to add to the ASC's StartingAbilities GameplayTagContainer when the AbilityMap is granted to the ASC, also in the spirit of modularity.

What do you think about that ?

Cheers !

reznok commented 7 months ago

I think it can be an Array, we already do that with Attributes, so it would function similarly.

A bool for "Add To Starting Abilities" (or similar) would make sense too.

👍

LeoFabre commented 7 months ago

Yeah that bool is a good idea.