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

fix(controller): Migrate from annotations to attributes #1983

Merged
merged 6 commits into from
Jul 24, 2024

Conversation

nickvergessen
Copy link
Member

No description provided.

Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
…he moment)

Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen force-pushed the bugfix/noid/migrate-to-attributes branch from f4309e1 to 342a990 Compare July 24, 2024 18:45
@@ -52,9 +53,6 @@ public function __construct(
}

/**
* @NoAdminRequired
* @NoCSRFRequired
Copy link
Member Author

Choose a reason for hiding this comment

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

its OCS, so this is included by default and not needed

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen merged commit 4f1716b into master Jul 24, 2024
42 checks passed
@nickvergessen nickvergessen deleted the bugfix/noid/migrate-to-attributes branch July 24, 2024 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants