Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.In admin/hosting/saas one form field asks for:
The nid of the db server to use.
Where can I find that?
| Comment | File | Size | Author |
|---|---|---|---|
| #17 | hosting_services-saas_form_select_platform_not_nid-2712701-17.patch | 1.77 KB | colan |
| #15 | hosting_services-get_saas_template_site_name_instead_of_the_nid-2712701-15.patch | 5.44 KB | gboudrias |
| #10 | ScreenClip.png | 31.7 KB | Yuri |
Comments
Comment #2
gboudrias CreditAttribution: gboudrias at Praxis Labs Coop commentedAs I replied through email, go to the servers tab, click on the server you want to use, then look at the nid in the URL of the edit tab. We should change this to autocomplete soon-ish.
Comment #3
gboudrias CreditAttribution: gboudrias at Praxis Labs Coop commentedComment #4
gboudrias CreditAttribution: gboudrias at Praxis Labs Coop commentedComment #5
colanI'll be working on this shortly, but feel free to steal it from me if you/anyone feels like working on it ASAP.
Comment #6
colanComment #7
colanLooking into this now.
Plan is to do the following:
Comment #8
colanHere's some code for the first one. It requires the patch over at #2720439: Add site autocompletion & ID retrieval without loading nodes so hopefully that gets in soon.
The one strange thing I had to do was overload the "nid" argument in the service call. In this case, it takes a site name like blah.example.com as the template site instead of the template site's node ID.
This is better for 2 reasons:
I didn't commit this yet in case it's a problem for anyone; if we need to do this differently. And we should also wait for that other patch to get in.
I'll commit code for the other two items above after I get to them.
Comment #9
colanComment #10
Yuri CreditAttribution: Yuri commentedI applied both required patched mentioned above, cleared cache but don't see any difference in the pages, apart from that the node ID's of the template site and database server are not recognized (while they are present).
Comment #11
Yuri CreditAttribution: Yuri commentedComment #12
colanThe first field now accepts only a site name (e.g. foo.example.com). That's what this patch does. So you'll need to enter that instead of a nid.
I'm not sure about the second field. What's showing up in your logs?
This is going to be reworked a bit (i.e. using another helper function) given the necessary changes for #2720439: Add site autocompletion & ID retrieval without loading nodes.
Comment #13
colanHere's an update with the latest patch from the other issue.
Comment #15
gboudrias CreditAttribution: gboudrias at Praxis Labs Coop commentedHad to reroll the patch, my recent commit was conflicting.
Comment #17
colanCommitted the above as the other patch has been merged.
Here's the code for the platform selection, which depends on #2721949: Deprecate _hosting_get_enabled_platforms(). Would someone kindly review that one?
Once we get this last bit of code merged, we can close this issue.
Comment #18
colanComment #20
colanDone!