Skip to content

Update acl.rst #3547

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

Closed
wants to merge 2 commits into from
Closed

Update acl.rst #3547

wants to merge 2 commits into from

Conversation

ruttencutter
Copy link
Contributor

Q A
Doc fix? yes
New docs? no
Applies to current
Fixed tickets

ACE is not defined in the scope of this document, hopefully this adds some clarity.

ACE is not defined in the scope of this document, hopefully this adds some clarity.
@ruttencutter ruttencutter reopened this Feb 5, 2014
@@ -90,6 +90,9 @@ Getting Started
Coming back to the small example from the beginning, you can now implement
ACL for it.

Once the ACL is created, you can grant access to objects by creating an Access Control Entity (ACE) to solidify the relationship between the entity and your user.


Copy link
Member

Choose a reason for hiding this comment

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

I like it!

Can you please break this onto multiple lines (after the first word that crosses the 72nd character) and remove one of the 2 new blank lines?

Thanks!

@weaverryan
Copy link
Member

Yes, you did! :) Thanks @iqfoundry!

weaverryan added a commit that referenced this pull request Feb 18, 2014
This PR was submitted for the master branch but it was merged into the 2.3 branch instead (closes #3547).

Discussion
----------

Update acl.rst

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | current
| Fixed tickets |
ACE is not defined in the scope of this document, hopefully this adds some clarity.

Commits
-------

cee5b9d Update acl.rst
37f49e3 Update acl.rst
@weaverryan weaverryan closed this Feb 18, 2014
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