Hi. There is a typo in the UI for the module description. I am creating a simple patch for this.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vishalkhialani created an issue. See original summary.

vishalkhialani’s picture

vishalkhialani’s picture

Assigned: vishalkhialani » Unassigned
Status: Active » Needs review

I've also made an update to the Name field in the info.yml file.

vishalkhialani’s picture

Oops, forgot to mention to kindly ignore the update to the name field if the original text "Registration" works as intended.

krina.addweb’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
27.21 KB
20.09 KB

@vishalkhialani Patch works well Thanks.

rootwork’s picture

I think it makes sense to commit the change including the title, so that it's clear this module is for a registration link, and not (for instance) handling registration itself. And, of course, it matches the title on the project page.

i-trokhanenko’s picture

Please add one line at end of file.

i-trokhanenko’s picture

Status: Reviewed & tested by the community » Needs work
andriansyah’s picture

Status: Needs work » Needs review
FileSize
499 bytes
200 bytes

newline added.

alonaoneill’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me!
Marking as RTBC
Thank you for working on the module!

i-trokhanenko’s picture

+1 RTBC

gisle’s picture

Priority: Normal » Minor

Changing priority.

i-trokhanenko’s picture

Status: Reviewed & tested by the community » Fixed

Thanks!

Status: Fixed » Closed (fixed)

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