@totten @seamuslee001 @colemanw @MikeyMJCO @twomice

* Rephrase now that it's in core
* Skip the steps for checking out a custom repo. The code now comes with core.
* Change example paths to be more readable
* Misc copyedit

* Rephrase now that the code is in core
* Fix some links (since the docs are now split from the code)
* Move "Key features" above "General design" since it's quicker to read

1.  Don't describe the `.use-civicrm-setup` protocol.  This is used in one
    place, and it's likely to get inverted in the near future.

2.  When customizing the model, most values shouldn't be passed directly
    into `init()`.  Let `init()` detect its defaults.  Then call
    `$setup->getModel()` and customize that.  (I had tried customizing via
    `init()` when first implementing `cv core:install` and found it less
    useful.  There was also a support issue when someone else tried the same
    thing.) This pattern should work more intuitively.

Co-Authored-By: Mikey O'Toole <mikey@mjco.uk>

Co-Authored-By: Mikey O'Toole <mikey@mjco.uk>

Removing reference to the word managed since it is confusing and mixes up entity types and entities

As per: https://github.com/civicrm/civicrm-core/pull/16529/files

Co-Authored-By: Mikey O'Toole <mikey@mjco.uk>

Co-Authored-By: Mikey O'Toole <mikey@mjco.uk>

This commit tries to stick to use the same naming-convention as the previous commit. I think it's still fair to consider what the convention should be, but this tries to present the current draft better.

I've kept this limited - if people agree we can merge this & then future PRs can add more as I think we probably want more recommended ones

Out of date even for v3, and supplanted by v4 architecture page.

colemanw authored 5 years ago and colemanw committed 5 years ago

Co-Authored-By: Mikey O'Toole <mikey@mjco.uk>