Skip to content

Add [AlsoNotifyCanExecuteFor] attribute #53

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

Merged
merged 4 commits into from
Dec 9, 2021

Conversation

Sergio0694
Copy link
Member

Contributes to #8

This PR adds the [AlsoNotifyCanExecuteFor] attribute.

API breakdown:

namespace CommunityToolkit.Mvvm.ComponentModel
{
    [AttributeUsage(AttributeTargets.Field, AllowMultiple = true, Inherited = false)]
    public sealed class AlsoNotifyCanExecuteForAttribute : Attribute
    {
        public AlsoNotifyCanExecuteForAttribute(string commandName);
        public AlsoNotifyCanExecuteForAttribute(string commandName, params string[] otherCommandNames);
        public string[] CommandNames { get; }
    }
}

@Sergio0694 Sergio0694 added feature 💡 A new feature being implemented need documentation 📃 This change needs a related documentation PR mvvm-toolkit 🧰 Issues/PRs for the MVVM Toolkit labels Dec 3, 2021
@Sergio0694 Sergio0694 force-pushed the dev/alsonotifycanexecutefor-attribute branch from 4d2f369 to 15e1ba4 Compare December 9, 2021 20:02
@Sergio0694 Sergio0694 merged commit ab75cb9 into main Dec 9, 2021
@delete-merged-branch delete-merged-branch bot deleted the dev/alsonotifycanexecutefor-attribute branch December 9, 2021 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 💡 A new feature being implemented mvvm-toolkit 🧰 Issues/PRs for the MVVM Toolkit need documentation 📃 This change needs a related documentation PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants