I installed the module, imported a list, and set it to be displayed on the register form.

What I see: I see a checkbox on the user registration form with no explanatory text.

What I expect to see: A form to subscribe to my mailing list.

If you check the box, the mailing list subscription information appears.

This could be solved by adding text to explain to check the box to see the mailing list subscription options, but if there is only one mailing list, it doesn't make a lot of sense to check a box to see the list, and then check a single list.

CommentFileSizeAuthor
#2 campaignmonitor_2941242-2_remove_states_api_logic.patch1.48 KBptmkenny
campaignmonitor_checkthebox.PNG7.1 KBptmkenny
campaignmonitor_nocheckthebox.PNG6.95 KBptmkenny

Comments

ptmkenny created an issue. See original summary.

ptmkenny’s picture

ptmkenny commented
Status: Active » Needs review
StatusFileSize
new campaignmonitor_2941242-2_remove_states_api_logic.patch1.48 KB

I suggest simply deleting the States API logic and letting users override it (in a custom module with hook_form_alter()) if they so desire.

  • New Zeal committed 24a55e5 on 8.x-1.x authored by ptmkenny 
    Issue #2941242 by ptmkenny: Register submodule has no interface text,...
NewZeal’s picture

NewZeal commented

Suggestion implemented and in 8.x-1.0-beta6

NewZeal’s picture

NewZeal commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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