Update EHR/NHANES with new fixes

@tony @aethelwulffe @yehster and @r0bby What will it take to update their version to the latest repo version so they will have the current fixes.

Not a server I have access to AFIK.

I canā€™t update it and you canā€™t update that leaves @tony and @yehster. Do we work on this project?

Just ask me ā€“ Iā€™ll probably get to it on Monday however. FYI:

1 Like

This has been completed. You might want to consider gitignoring your sqlconf file ā€“ or using dotenv to store actual values. The composer files also cause the repo to become dirty ā€“ meaning I have to stash changes before doing anything ā€“ this isnā€™t good. FYI

The config file is not actually ā€œconfiguredā€ and is necessary at the moment for the installer, but the composer crap spammed all over the place is useless and messy junk, like a lot of other junk that has been carelessly allowed to junk up the code base.

1 Like

Two things: 1 the sqlconf.php and other ā€œconfigurationā€ files should, in fact be generated by the setup (no template needed) and .gitignored

Long term the most of ā€˜sitesā€™ directory (or any dir that gets written to with a need for persistence) should NOT be part of the repo. If we ever want to containerize this we have to have clean separation for the parts.

  1. Application Code
  2. Configuration setting
  3. Database
  4. Persistent Storage (uploaded Docs, local templates, site menu changes, etc)
  5. Add-on Modules

You should be able to delete/replace #1 and #5 at anytime with a new container/codeset with out effecting the installation.

1 Like

Yes. Let us try this experiment! We will call itā€¦ā€œThe Common Sense Releaseā€!

2 Likes