Problem/Motivation

Reviewing and testing merge requests for this project module requires cloning issue fork branches locally and installing the module in some sort of local development environment.

Proposed resolution

Enable Tugboat live previews for merge requests on this project.

Remaining tasks

Add Tugboat configuration to the module repository.

Issue fork google_cse-3265745

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

joegraduate created an issue. See original summary.

joegraduate opened merge request !3

joegraduate’s picture

joegraduate
Primary language English
Location Arizona, USA
commented
Status: Active » Needs review
gravelpot’s picture

gravelpot commented
Status: Needs review » Needs work

Hi @joegraduate, sorry for the long delay on this. We are doing an issue queue review on this module and this looks like something we should go ahead and merge.

Since so much time has passed, and we aren't actively using Tugboat so aren't familiar with the syntax, do you want to do one more quick pass to make sure this is still what you need?

joegraduate’s picture

joegraduate
Primary language English
Location Arizona, USA
commented
Status: Needs work » Needs review

Hi @gravelpot. I just looked over the proposed tugboat config file in the MR and I think everything still looks good. The file included in the MR was adapted from the example on this documentation page.

The only changes I made to the example config were these:

  • Used the Drupal 9.3 version of the q0rban/tugboat-drupal docker image instead of 9.0.
  • Used google_cse as the module name instead of webform.

I'm assuming we'd want to use Drupal core version 9.3.x to build the previews currently (until 9.4.0 is released).

gravelpot’s picture

gravelpot commented
Assigned: Unassigned » mmarler

Thanks @joegraduate, we're going to work on getting this rolled out in a new release next week.

mmarler made their first commit to this issue’s fork.

  • mmarler committed c4cf17c on 8.x-3.x 
    Issue #3265745: Enable Tugboat live previews
mmarler’s picture

mmarler commented
Status: Needs review » Fixed

This has been merged in and we have released a new version that adds tugboat integration.

https://www.drupal.org/project/google_cse/releases/8.x-3.7

Status: Fixed » Closed (fixed)

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