Add a config form

I can work on it.

Hi I added a config form please revise.

Comments added to the MR, we also need a test because currently it is broken.

Hi I fixed the errors please revise.

I would suggest using a different form ID than big_menu_form_id_settings. It seems wrong to put "form_id" in the ID. How about bigmenu_settings (the module name doesn't have a space in it).

Similarly, in the routing file, it should be bigmenu.settings, not big_menu.settings (the module name doesn't have a space in it).

Then, we should add configure: bigmenu.settings to the info file.

Hi I did the requested changes please revise.

The code looks good to me, but I can't get the patch to apply when I tried using Composer. I don't know why.

Also we should fix the Composer errors that show for the tests.

you can try to clone the module

Still needs tests.

is there something I need to do?

Hi I install the module using composer after that ran the issue fork's commands and checkout to respective branch.

I was able to revised the changes using composer to install the module.

per #11, this has to stay Needs Work until tests are added.

I will work on it.

Hi I added a test to this new feature please revise

Testing Drupal\Tests\bigmenu\Functional\BigMenuSettingsFormTest
.                                                                   1 / 1 (100%)

Time: 00:13.657, Memory: 10.00 MB

OK (1 test, 10 assertions)

Test is passing

Thanks

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