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 LimeSurvey account if yunohost user is deleted #13

Open
zamentur opened this issue Feb 9, 2017 · 4 comments
Open

Delete LimeSurvey account if yunohost user is deleted #13

zamentur opened this issue Feb 9, 2017 · 4 comments
Labels
Projects

Comments

@zamentur
Copy link
Contributor

zamentur commented Feb 9, 2017

Currently I just warn the yunoadmin to delete the user account in LimeSurvey

If the user is not the administrator we should delete it.

I would delete it via console.php command, but there is no commands to do that.

So a new LimeSurvey command should be created and added with a patch
Like the UpdateCommand we use to migrate DB https://github.com/YunoHost-Apps/limesurvey_ynh/blob/master/patches/app-0001-7ca4932.patch

@Shnoulle (or a LimeSurvey contrib) May be you are able to create a command like this ?

@zamentur zamentur created this issue from a note in General (In Progress) Feb 9, 2017
@zamentur zamentur added the bug label Feb 9, 2017
@zamentur zamentur moved this from In Progress to TODO in General Feb 9, 2017
@Shnoulle
Copy link
Contributor

Shnoulle commented Feb 9, 2017

I think of creating a yunohost plugin.

Then can use
php application/command/console.php plugin cron myaction

Or did you prefer a new controller ?

Like you want, maybe best : UpdateCommand move it to YnhCommand and we can add desired function inside .
And follow LS api (with specific release of the patch in github).
=> I start for this

  • disableUser => disable auth access for a user
  • enableUser => enable auth access (and create iof didn't exist at this time)

Ok ?

@Shnoulle
Copy link
Contributor

Shnoulle commented Feb 9, 2017

PS : for next release : what LS release did you take ? https://github.com/LimeSurvey/LimeSurvey/releases/tag/2.62.1%2B170130 ?

@Shnoulle
Copy link
Contributor

WIP : https://github.com/Shnoulle/LimeSurvey-Shnoulle/tree/master_ynhversion

Finnally : a ynh plugin for managing user and maybe some other part if needed after.

The Allow update by CLI is already in 3.0
The Activate a plugin by command are surely in a future release (3.0).

I see to update on next release with

  • deactivate user access (this issue)
  • upload dir to home in the plugin
  • Maybe a system to allow user to update some part of config in home directory ?

@Shnoulle
Copy link
Contributor

Just : some delay , some other work to do (and some test on GnuSocial ). But i keep the project

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

No branches or pull requests

2 participants