Skip to content
This repository has been archived by the owner on Oct 8, 2020. It is now read-only.

Document Financial Permissions #253

Merged
merged 3 commits into from
Mar 3, 2018
Merged

Document Financial Permissions #253

merged 3 commits into from
Mar 3, 2018

Conversation

aydun
Copy link
Contributor

@aydun aydun commented Mar 1, 2018

Document Financial Permissions - see #241

More consistent formatting of permission names

@seancolsen
Copy link
Contributor

Nice additions! And thanks for splitting up the commits like that 🙂

I made some edits that I'll admit are a bit nit-picky. Just trying to make the text faster to read. I'm going to mere this now, but I'm happy to discuss my edit more if there's anything you disagree with.

@seancolsen seancolsen merged commit d7d7e17 into civicrm:master Mar 3, 2018
@aydun
Copy link
Contributor Author

aydun commented Mar 3, 2018

Thanks @seanmadsen - all good improvements. Personally I like examples early - I grasp the potential usefulness of "see events payments but not donations" more easily than "control access by financial type" (what was a 'financial type' anyway??). But then I skip to the bottom of man pages to read the examples before wading through the option details!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants