Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Delete Account Option #4065

Merged
merged 1 commit into from
Feb 21, 2020
Merged

Delete Account Option #4065

merged 1 commit into from
Feb 21, 2020

Conversation

LaocheXe
Copy link
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
Contributor

Jimmi08 commented Jan 11, 2020

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

@LaocheXe
Copy link
Member Author

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

@Jimmi08
Copy link
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
Member

CaMer0n commented Feb 21, 2020

Thank you @LaocheXe ! (and for your patience)

@Moc
Copy link
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