You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The @FeatureVariant annotation from quarkus-unleash, which is meant to retrieve the variant from a given toggle, requires a name attribute. Unless I missed something, there is currently no indication that the annotation requires the toggle name and not the variant name. If the variant name is provided (which is how I tried to use it initially), Unleash won't return any data.
My suggestion would be to introduce a new FeatureVariant#toggleName attribute, deprecate FeatureVariant#name immediately and remove it eventually. The annotation could also be better documented with Javadoc and the extension doc.
Finally, for the sake of consistency if that is implemented, FeatureToggle#toggleName could be considered as well to replace FeatureToggle#name.
The text was updated successfully, but these errors were encountered:
gwenneg
changed the title
The 'name' attribute of @FeatureToggle is confusing
The 'name' attribute of @FeatureVariant is confusing
Mar 6, 2024
In Unleash, both toggles and variants have a
name
.The
@FeatureVariant
annotation from quarkus-unleash, which is meant to retrieve the variant from a given toggle, requires aname
attribute. Unless I missed something, there is currently no indication that the annotation requires the toggle name and not the variant name. If the variant name is provided (which is how I tried to use it initially), Unleash won't return any data.My suggestion would be to introduce a new
FeatureVariant#toggleName
attribute, deprecateFeatureVariant#name
immediately and remove it eventually. The annotation could also be better documented with Javadoc and the extension doc.Finally, for the sake of consistency if that is implemented,
FeatureToggle#toggleName
could be considered as well to replaceFeatureToggle#name
.The text was updated successfully, but these errors were encountered: