Skip to content

Delete Account Option#4065

Merged
CaMer0n merged 1 commit intoe107inc:masterfrom
LaocheXe:master
Feb 21, 2020
Merged

Delete Account Option#4065
CaMer0n merged 1 commit intoe107inc:masterfrom
LaocheXe:master

Conversation

@LaocheXe
Copy link
Copy Markdown
Member

@LaocheXe LaocheXe commented Jan 10, 2020

Delete Account Option added in admin area. Also updated the default_install.xml to add missing core options. - Regarding: #3945 and #3946

This gives the admin an option to display/hide the delete account button in user settings area. Also added del_accu, and other missing user management SitePref settings to default_install.xml which are:
avatar_upload
photo_upload
profile_rate
profile_comments
force_userupdate
del_unv
signature_access

Also did a test install and had no issues.

Delete Account Option added in admin  area. Also updated the default_install.xml to add missing core options.
@Jimmi08
Copy link
Copy Markdown
Contributor

Jimmi08 commented Jan 11, 2020

Please, add the name of the delete user event to your description.

@LaocheXe
Copy link
Copy Markdown
Member Author

@Jimmi08 it give an option to show or hide the Delete account button.

@Jimmi08
Copy link
Copy Markdown
Contributor

Jimmi08 commented Jan 11, 2020

I know, but this option shouldn't be available without this event trigger. Maybe it exists already.

@CaMer0n CaMer0n merged commit e4b5f06 into e107inc:master Feb 21, 2020
@CaMer0n
Copy link
Copy Markdown
Member

CaMer0n commented Feb 21, 2020

Thank you @LaocheXe ! (and for your patience)

@Moc
Copy link
Copy Markdown
Member

Moc commented Jun 3, 2020

I know, but this option shouldn't be available without this event trigger. Maybe it exists already.

Added as a reminder for now. #4184
Will add asap

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants