Skip to content
Snippets Groups Projects

sysadmin#3 - Rework "Get the code" and "Run the installer"

sysadmin#3 - Rework "Get the code" and "Run the installer"
All threads resolved!
Merged totten requested to merge totten/installation:master-setup-wf into master
All threads resolved!

This is primarily motivated by sysadmin#3 (closed) - i.e. driving toward the new installer UI on D7/BD. But it goes a bit further to also update D8/D9.

General themes:

  • Use more collapsible admonitions (with more explanatory headings) to address variations/edge-cases/details
  • Try to keep the main body of each major section to a couple mini-paras.
  • In similar CMS pages (D7<=>BD; D8<=>D9), provide similar organization of subsections
  • Consolidate the discussion about using separate DBs. This is an optional topic (more appropriate to larger deployments - less appropriate to smaller/evaluator deployments), and each page should hyperlink it.

It may be a little difficult to read the diffs -- probably easier to either:

  1. Go to temporary site: http://spades.undo.it:8008/backdrop/, http://spades.undo.it:8008/drupal7/, http://spades.undo.it:8008/drupal8/, http://spades.undo.it:8008/drupal9/

  2. Checkout the repo and run mkdocs serve, eg

docker run --rm -v "$PWD:/docs" -p 8000:8000 -w /docs "mjcoltd/civicrm-docker-mkdocs" serve --dirtyreload -a 0.0.0.0:8000

Ping @MikeyMJCO @seamuslee @herbdool

Edited by totten

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • seamuslee
  • totten added 16 commits

    added 16 commits

    • 8f87b565...b6fc0495 - 6 commits from branch documentation/docs:master
    • b8d6886b - Tweak introductory admonitions. Use collapsing better.
    • 75c3f389 - Backdrop, D7 - Remove section "Identify the database"
    • dc00e4ee - Requirements - Add a subsection under "MySQL" discussing the need for connection URL
    • 626b0f2d - Add placeholder page for cv installer
    • 89edfded - Update section "Run the installer" (D7/BD)
    • fc65f33f - Update section "Run the installer" (D8/D9). Extract CLI.
    • e883e16c - Update section "Get the code" (D7/BD). Use collapsible subsections.
    • 19e329df - Update section "Get the code" (D8/D9). Use collapsible subsections.
    • 850483f5 - Port changes from !16 (merged)
    • 1c033632 - Update section "More detail: Compilation tasks" (D8/D9)

    Compare with previous version

  • totten resolved all threads

    resolved all threads

  • Author Owner

    I've rebased on the current master. There were some conflicts with !16 (merged), so I copied over the new admonition (and then copy-edited).

  • homotechsual approved this merge request

    approved this merge request

  • I'm happy with these changes/this direction of travel :-)

  • totten added 2 commits

    added 2 commits

    • 2ac6629c - Copyedit "Get the code" (D8/D9)
    • 7b08a62a - cli-cv.md - Rewrite as standalone page

    Compare with previous version

  • Author Owner

    Thanks @MikeyMJCO. :) Merging.

  • merged

  • totten mentioned in commit e0dff2c9

    mentioned in commit e0dff2c9

  • Please register or sign in to reply
    Loading