It would be useful to be able to get access to properties like web server, publish path, and and platform status in site nodes.

I do a major JOIN in devshop to load all of this information into one object to provide a better UI that gives you all the info.

Let's do a quick JOIN on platforms to load web_server, publish_path, platform_verified, and platform_status into site nodes.

CommentFileSizeAuthor
#3 2881425-platform-props-in-sites.patch1.02 KBjon pugh

Comments

Jon Pugh created an issue. See original summary.

  • Jon Pugh committed 30ada2c on 2881425-platform-props-in-sites 
    Issue #2881425: Inject platform properties into site nodes: web_server,...
jon pugh’s picture

jon pugh
he/him
Primary language English
Location Newburgh, NY
commented
StatusFileSize
new 2881425-platform-props-in-sites.patch1.02 KB
jon pugh’s picture

jon pugh
he/him
Primary language English
Location Newburgh, NY
commented

This is leading up to me creating a better list of servers on site nodes, which will allow us to put a new method on HostingService classes for site_form() and platform_form().

...Which will lead to better Hosting HTTPS :)

  • Jon Pugh committed 4011ce4 on 2881425-platform-props-in-sites 
    Issue #2881425: Load web and database server nodes into a $node->servers...
colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
commented

Would be nice to break up that line:

SELECT ...
AS ...
...
jon pugh’s picture

jon pugh
he/him
Primary language English
Location Newburgh, NY
commented

I 100% agree, and was tempted, but I wanted to keep the change small.

Coming soon...

  • helmo committed 850a19c on 2881425-platform-props-in-sites 
    Issue #2881425 by helmo: break up long query line

  • Jon Pugh committed 30ada2c on 7.x-3.x 
    Issue #2881425: Inject platform properties into site nodes: web_server,...
  • Jon Pugh committed 4011ce4 on 7.x-3.x
    Issue #2881425: Load web and database server nodes into a $node->servers...
  • helmo committed 850a19c on 7.x-3.x
    Issue #2881425 by helmo: break up long query line
  • helmo committed f700916 on 7.x-3.x
    Issue #2881425 by Jon Pugh: Merge branch '2881425-platform-props-in-...
helmo’s picture

helmo commented
Status: Needs review » Fixed

Merged. And I've wrapped the long line to make it a bit more readable.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.