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

feat: on action decorator #1090

Draft
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

sveneberth
Copy link
Member

Usage

class MyModule(List):
    @on_edited
    @on_added
    def entry_observer(self, skel):
        print(f"Entry changed: {skel = }")

What do u think of this concept?

@sveneberth sveneberth added the feature New feature or request label Mar 5, 2024
@sveneberth sveneberth added this to the Action-Skels milestone Mar 5, 2024
@sveneberth sveneberth changed the title Feature/on action feat: on action decorator Mar 5, 2024
@phorward
Copy link
Member

phorward commented Mar 5, 2024

What do u think of this concept?

Quite interesting. This looks like a more dynamic replacement for #972.
But I would like to combine it with the meaning of actions. So an action is defined once, and generates hooks, access, access-right etc. via one definition.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

2 participants