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

Remove dead link and update outdated link #95

Open
wants to merge 1 commit into
base: master
from

Conversation

@jonathan-s
Copy link

@jonathan-s jonathan-s commented Apr 26, 2019

No description provided.

Copy link
Owner

@dfunckt dfunckt left a comment

Thanks for the PR!

I'm fairly certain the internal link used to work before -- the blank .. _authorization in Django: syntax used to create an anchor we could link to -- but apparently not anymore. Its purpose was to link to the Using Rules with Django headline and I think it's a useful cross-reference to preserve, hence my comment.

@@ -201,7 +201,7 @@ Predicates can do pretty much anything with the given arguments, but must
always return ``True`` if the condition they check is true, ``False``
otherwise. ``rules`` comes with several predefined predicates that you may
read about later on in `API Reference`_, that are mostly useful when dealing
with `authorization in Django`_.
with authorization in Django.

This comment has been minimized.

@dfunckt

dfunckt Apr 28, 2019
Owner

Can you please make this point to the Using Rules with Django headline?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.