Modify

Opened 6 years ago

Closed 4 years ago

Last modified 3 years ago

#3153 closed enhancement (fixed)

[patch] Easy option to disable email verification

Reported by: coderanger Owned by: hasienda
Priority: normal Component: AccountManagerPlugin
Severity: normal Keywords: user email verifiy
Cc: heikki Trac Release: 0.11

Description

Attached patch adds a simple option to enable or disable email verification, as well as never trying to verify the address of an admin (so they can't get locked out of webadmin).

Entirely untested, but it is late and I am going to bed.

Attachments (1)

verify_email_option.patch (3.2 KB) - added by coderanger 6 years ago.

Download all attachments as: .zip

Change History (9)

Changed 6 years ago by coderanger

comment:1 Changed 6 years ago by coderanger

also I set the default to be to not verify addresses, as this is the current behavior of acct_mgr.

comment:2 Changed 6 years ago by s0undt3ch

  • Owner changed from mgood to s0undt3ch
  • Status changed from new to assigned

This can probably be marked a duplicate of #442 since what you address here is a fix for what's addressed on that ticket.

comment:3 Changed 6 years ago by s0undt3ch

  • Owner changed from s0undt3ch to mgood
  • Status changed from assigned to new

Dam! It's the 2nd time that when I comment a ticket, I wrongly become the owner of the ticket. Reverting it back.

comment:4 Changed 6 years ago by heikki

  • Cc hjtoi-bugzilla@… added; anonymous removed

comment:5 Changed 6 years ago by heikki

  • Cc heikki added; hjtoi-bugzilla@… removed

comment:6 Changed 4 years ago by hasienda

  • Keywords user email verifiy added
  • Owner changed from mgood to hasienda
  • Summary changed from Easy option to disable email verification to [patch] Easy option to disable email verification

Nice, certainly a reasonable request, as the admin web UI is already quit sophisticated, and it'll make a nice follow-up on [9277] as well, as you don't need to give access to plugin manager for activation/deactivation of the EmailVerificationModule.

comment:7 Changed 4 years ago by hasienda

  • Resolution set to fixed
  • Status changed from new to closed

(In [9304]) AccountManagerPlugin: Add easy option to switch email verification from admin UI, closes #3153.

This is a follow-up to changeset [9277] as we introduce an easy way to enable/disable this feature while keeping the new default behavior of making email address verification mandatory on activation of EmailVerificationModule. I've taken the chance for some cosmetic changes as well, especially to remove radio buttons and use checkboxes consistently for all options.

comment:8 Changed 3 years ago by hasienda

(In [10519]) AccountManagerPlugin: Make option verify_email effective for RegistrationModule too, refs #3153, #3989, #5509 and #9051.

Only module state (enabled/disabled) has been checked before, when deciding on the email address field being optional vs. required since changeset [9304].

Add Comment

Modify Ticket

Action
as closed The owner will remain hasienda.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.