Skip to content

feat(actix-web): add contains, all, any to AuthorityGuard#129

Open
frederik-uni wants to merge 3 commits into
DDtKey:mainfrom
frederik-uni:frederik-uni-patch-1
Open

feat(actix-web): add contains, all, any to AuthorityGuard#129
frederik-uni wants to merge 3 commits into
DDtKey:mainfrom
frederik-uni:frederik-uni-patch-1

Conversation

@frederik-uni

Copy link
Copy Markdown

Description:

Instead of AuthorityGuard::new its now named AuthorityGuard::contains & AuthorityGuard::any & AuthorityGuard::all were added

Checklist:

  • Tests for the changes have been added (for bug fixes / features);
  • Docs have been added / updated (for bug fixes / features).

@DDtKey DDtKey changed the title add AuthorityGuard::contains, all, any feat(actix-web): add contains, all, any to AuthorityGuard Mar 23, 2025

@DDtKey DDtKey left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thank you for the contribution 🙏

I’ve left a couple of minor comments, please take a look.

Comment thread actix-web-grants/src/guards.rs
Comment thread actix-web-grants/src/guards.rs Outdated
@DDtKey

DDtKey commented Mar 31, 2025

Copy link
Copy Markdown
Owner

Could you apply rustfmt with nightly flag?

Comment thread actix-web-grants/src/guards.rs Outdated
Co-authored-by: Artem Medvedev <i@ddtkey.com>
@DDtKey

DDtKey commented Apr 6, 2025

Copy link
Copy Markdown
Owner

Could you apply rustfmt and LGTM 👍

Thank you for the contribution 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants