Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.2k
New Data Voter Article (continuation)#3594
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
| .. seealso:: | ||
| Voters can also be used in other ways, like, for example, blacklisting IP |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
"like, for example" aren't that 2 words saying the same? I would use either "like" or "for example".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I like it the way it is.
This PR was merged into the 2.3 branch.Discussion----------New Data Voter Article (continuation)Hiya guys!This is just a continuation of#3138 by@monbro. I needed to rebase it against the 2.3 branch (which had conflicts), and wanted to proofread it (hence the fresh PR).Please check out my latest commit to see if I've made any mistakes :).| Q | A| ------------- | ---| Doc fix? | no| New docs? | yes| Applies to | 2.3| Fixed tickets |#2877Thanks!Commits-------d3f9383 [#3594] Nice tweaks thanks to@wouterj and@xabbuh2391758 [#2877][#3138] Proofreading the new voter data permission entry11aead7 simplified the example8227270 updated according to the reviewda7b97e missed one comment9b91501 updated the docs according to the last review872a05f updated the link from ACL to the data permission voters1fd3b0e updated docs according to the review5275230 updated with missing fixesf4eb5f3 updated docs according to the reviewsee0def1 improved tip box with additional link to /cookbook/security731dcad updated page with suggestion from the review1466fa7 improvements according to the reviews99b1b0f a couple of changes according to the comments, not finished now2bda150 create voters_data_permission.rst article
Thanks a lot guys! I made all of the changes you suggested and I'm very very happy to have this guy merged into the docs! And also a big thanks to@monbro, who did all of the work :). |
Hiya guys!
This is just a continuation of#3138 by@monbro. I needed to rebase it against the 2.3 branch (which had conflicts), and wanted to proofread it (hence the fresh PR).
Please check out my latest commit to see if I've made any mistakes :).
Thanks!