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

Add #has_strict_role? documentation #498

Merged
merged 2 commits into from
Feb 27, 2019
Merged

Conversation

maerch
Copy link
Contributor

@maerch maerch commented Nov 22, 2018

Couldn't find any documentation to check for an exact role for a single query without enabling the strict mode.

Found this method in the source code which could be handy for others as well.

@coveralls
Copy link

Coverage Status

Coverage increased (+1.0%) to 93.284% when pulling 769ddd0 on maerch:patch-1 into 522c0d7 on RolifyCommunity:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+1.0%) to 93.284% when pulling 769ddd0 on maerch:patch-1 into 522c0d7 on RolifyCommunity:master.

@coveralls
Copy link

coveralls commented Nov 22, 2018

Coverage Status

Coverage remained the same at 92.516% when pulling a24d8d1 on maerch:patch-1 into 199ca1e on RolifyCommunity:master.

Copy link
Member

@EppO EppO left a comment

Choose a reason for hiding this comment

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

LGTM

@EppO EppO merged commit bb3d445 into RolifyCommunity:master Feb 27, 2019
@EppO
Copy link
Member

EppO commented Feb 27, 2019

Thanks for you PR!

@EppO EppO added this to the rolify 5.3 milestone Feb 28, 2019
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.

3 participants