Could you change the following text on the drupal_origins_security_help function...?

$output .= '

Drupal Origins i18n Extension

';

/**
 * Implements hook_help().
 */
function drupal_origins_security_help($path, $arg) {
  switch ($path) {
    case 'admin/help#drupal_origins_security':
      $module_path = drupal_get_path('module', 'drupal_origins_security');
      $output = '';
     $output .= '<h3>Drupal Origins i18n Extension</h3>';
    $output .= '<p>Drupal Origins Security & Legal extension

Thank you!

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lcontreras created an issue. See original summary.

lcontreras’s picture

Issue summary: View changes
lcontreras’s picture

Assigned: Unassigned » lcontreras
Status: Active » Needs review
FileSize
883 bytes

I'll add a new patch for this issue.

dhruveshdtripathi’s picture

Status: Needs review » Needs work

The help page does not follow the Help text standards. https://www.drupal.org/docs/develop/documenting-your-project/help-text-standards

lcontreras’s picture

Thanks for reviewed this. I'll use the following link like an example:

http://cgit.drupalcode.org/examples/tree/page_example/page_example.modul...

Is that ok?.

dhruveshdtripathi’s picture

lcontreras’s picture

Status: Needs work » Needs review
FileSize
2.26 KB

OK. So, I created a new patch following the help page standars.

Could you take a look please?

Thank you very much.

joe_carvajal’s picture

Status: Needs review » Fixed
Issue tags: +DevDaysSeville

Looks good for me.
Commited to the 7.x branch.

Thank you!

lcontreras’s picture

Status: Fixed » Closed (fixed)

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