- Issue created by @Nishant
- Status changed to Needs work
over 1 year ago 7:20pm 9 May 2023 - ๐ฎ๐นItaly apaderno Brescia, ๐ฎ๐น
The issue summary should always describe what should be fixed/changed. The title is not sufficient to describe what the issue is, even in the case a patch is provided.
- ๐ฎ๐นItaly apaderno Brescia, ๐ฎ๐น
Since the content of the walkme.info.yml file is the following, it is not clear which lines should be unnecessary.
name: WalkMe Integration type: module description: 'Very basic module for implementing WalkMeโข within Drupal.' core_version_requirement: ^8 || ^9 || ^10 package: WalkMe configure: admin/config/system/walkme
- @nishant opened merge request.
- ๐ฎ๐ณIndia Nishant
@apaderno,
This line is not required in info.yml 'admin/config/system/walkme'
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 5:30am 10 May 2023 - Status changed to Needs work
over 1 year ago 5:59am 10 May 2023 - ๐ฎ๐ณIndia Shiv_Sharma
@nishant
arrtibutes are correct issue is we are using URL as value.
configure: admin/config/system/walkme
It should be routename
configure: walkme.walkme_form
- ๐ฎ๐ณIndia shashank5563 New Delhi
@nishant, Please see the .info.yml file documentation.
https://www.drupal.org/docs/develop/creating-modules/let-drupal-know-abo... โ
- @nishant opened merge request.
- Status changed to Needs review
over 1 year ago 6:32am 10 May 2023 - First commit to issue fork.
- Status changed to Fixed
over 1 year ago 1:45pm 10 May 2023 Automatically closed - issue fixed for 2 weeks with no activity.