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

Add del command to remove emails from list in sympa.pl #911

Merged
merged 1 commit into from
Oct 5, 2021

Conversation

kukoarmas
Copy link
Contributor

In the sympa.pl CLI we have the "import" option to add subscribers to a list, but we lack a similar command to remove subscribers from the list.
This command is useful if you need to remove subscribers from a batch script

@ikedas
Copy link
Member

ikedas commented Mar 14, 2020

For me, this PR seems related to #901/#902. IMHO --add and --del (or maybe --add_member and --del_member) would be better to be implemented.

@kukoarmas
Copy link
Contributor Author

For me, this PR seems related to #901/#902. IMHO --add and --del (or maybe --add_member and --del_member) would be better to be implemented.

Do you mean that I should implement those two commands? If I implement --add_member for multiple entries, wouldn't it duplicate the --import functionality?

I'm ok implementing both commands, I'm just asking ;)

@ikedas
Copy link
Member

ikedas commented Mar 28, 2020

Do you mean that I should implement those two commands? If I implement --add_member for multiple entries, wouldn't it duplicate the --import functionality?

No you may not, if you will wait for the work by @dverdin progressing.

N.B. I think the “--import“ option On CLI was introduced for consistency with “Multiple add” function on web GUI. The “del” function looks not having multiple counterpart.

ikedas added a commit that referenced this pull request Oct 5, 2021
… kukoarmas & ikedas

Add del command to remove emails from list in sympa.pl (#911)
@ikedas ikedas merged commit b1308d0 into sympa-community:sympa-6.2 Oct 5, 2021
@ikedas ikedas changed the title Add unsubscribe command to remove emails from list in sympa.pl Add del command to remove emails from list in sympa.pl Oct 5, 2021
@ikedas ikedas added this to the 6.2.68 milestone Oct 5, 2021
@ikedas
Copy link
Member

ikedas commented Oct 5, 2021

@kukoarmas , sorry for delayed merge. As @dverdin 's work got stuck, this PR was merged first.

The name of command was changed to "del" standing for the action by admins to delete any subscribers.

Thank you for your contribution!

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

Successfully merging this pull request may close these issues.

2 participants