Contribution Date
Technology
Contribution Project
Contribution Details
Hi,
manual Review :
.info file
1) We must not enter 'version' into info file , Drupal does that automatically for you during release. Please read here https://www.drupal.org/node/542202#version
.install file
Suggestions :
1) if your path "/admin/structure/services/list/" is an configuration path then you can define it inside your .info file with configure parameter.
2) You should try writing drupal_set_message() inside hook_enable() rather than in hook_install().
.module file
1) I guess this should be translatable string 'name' => 'Swagger UI'
inside hook_libraries_info() ?
Contribution Issue Link
Contribution Link
Files count
0
Patches count
0