starshot-prototype icon indicating copy to clipboard operation
starshot-prototype copied to clipboard

Fix errors on Status report

Open TravisCarden opened this issue 9 months ago • 2 comments

There are a number of errors and warnings on the Status report on a fresh installation. We should fix as many as we can.

TravisCarden avatar May 05 '24 04:05 TravisCarden

  1. The entity/field definitions being out of date is a definite bug and we need to fix that. I'm guessing we might be missing some configuration in our recipes.
  2. The trusted host settings are probably something we should be setting up for local development, like automatically supplying a settings.local.php with trusted_host_patterns including 127.0.01 and localhost (along with the Lando and MAMP versions mentioned in the d.o documentation). We also probably want to update our CI to ensure that we have a trusted host pattern set up for Tugboat.
  3. The Webform problem requires us to solve the "how do you get JavaScript libraries into a Drupal site?" question, which is a long-standing pain point. We might want to consult @lauriii for ideas there; one possibility is to take advantage of npm and supply the libraries in a package.json instead of using hacks like Asset Packagist.

phenaproxima avatar May 06 '24 16:05 phenaproxima

  • Fixed trusted_host_patterns error on Tugboat: https://github.com/phenaproxima/starshot-prototype/commit/cdb478a4f52d1df25db01c47369d32fddbdbfb8f.
  • Fixed rebuild access enabled error on Tugboat: https://github.com/phenaproxima/starshot-prototype/compare/5414fa9a...c1802404.
  • Turns out Webform module provides a Drush command to install the external libraries. Fixed in 48a22a0e60ee2064dca6f30122ba3d72d7e4545d. (Note: That takes care of the final error on Tugboat.)

TravisCarden avatar May 10 '24 21:05 TravisCarden