Follow-up to #2827845: [roadmap] YAML Form 8.x-1.x to Webform 8.x-5.x

Problem/Motivation

The YAML Form module is going to be moving into the 8.x-5.x branch of the Webform module.

The YAML Form module namespace must be converted to use the Webform namespace.

Proposed resolution

Remaining tasks

Notes

  • Sandboxes don't support automated tests.

Renaming

  • YamlForm => Webform
  • yamlform => webform
  • YAML Form => Webform
  • Form => Webform
  • form => webform
CommentFileSizeAuthor
#5 rename_yaml_form-2827847-5.patch15.21 KBjrockowitz
#4 Test result YAML Form Demo Site.png374.54 KBjrockowitz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jrockowitz created an issue. See original summary.

  • jrockowitz committed e439a76 on 2827847-yamlform-to-webform-module 
    Issue #2827847: Rename YAML Form 8.x-1.x to Webform 8.x-5.x

  • jrockowitz committed 850570e on 2827847-yamlform-to-webform-module 
    Issue #2827847: Rename YAML Form 8.x-1.x to Webform 8.x-5.x
jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer commented
Status: Active » Fixed
FileSize
Test result YAML Form Demo Site.png374.54 KB

So I wrote a drush yamlform-to-webform-convert command that automatically converts the YAML Form module to the Webform. Making the renaming automated will allow me to continue working on the YAML Form code base while I #2827850: Provide YAML Form 8.x-1.x to Webform 8.x-5.x migration (yamlform_to_webform.module)

Below are the commands required to convert the YAML form module to the Webform module

First, you must make the yamlform_to_webform.module visible by adding $settings['extension_discovery_scan_tests'] = TRUE to your settings.php.

drush en -y yamlform_to_webform
drush yamlform-to-webform-convert
drush en -y webform

...and all tests are passing.

DO NOT USE THE GENERATED WEBFORM MODULE FOR ANYTHING OTHER THAN TESTING.

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer commented
Status: Fixed » Needs review
FileSize
rename_yaml_form-2827847-5.patch15.21 KB

Attached is the patch which just needs to pass the automated test and then I will commit it.

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer commented
Status: Needs review » Needs work
jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer commented
Status: Needs work » Needs review
jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer commented
Status: Needs review » Fixed

I can't get the automated tests to run.

  • jrockowitz committed 6b81d12 on 8.x-1.x 
    Issue #2827847 by jrockowitz: Rename YAML Form 8.x-1.x to Webform 8.x-5....

Status: Fixed » Closed (fixed)

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