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

list description with SOAP #1542

Closed
acasadoual opened this issue Dec 5, 2022 · 8 comments · Fixed by #1545
Closed

list description with SOAP #1542

acasadoual opened this issue Dec 5, 2022 · 8 comments · Fixed by #1545
Milestone

Comments

@acasadoual
Copy link
Contributor

Possibility to consult the description of the list

Possible Solution

a) Extend in /etc/sympa/sympa.wsdl
b) Extend "SOAP type listType" in /usr/share/sympa/lib/Sympa/WWW/SOAP.pm
c) Extend "sub info" in /usr/share/sympa/lib/Sympa/WWW/SOAP.pm
d) Use $list->get_info
e) systemctl restart sympasoap

Context

Attach patch for 6.2.68
description.patch.zip

@ikedas
Copy link
Member

ikedas commented Dec 5, 2022

@acasadoual , if possible, could you please submit the proposed changes as a Pull Request? If that is too difficult, let me know and I will submit it for you.

@acasadoual
Copy link
Contributor Author

Yes, i have submitted these Pull Requets.

@ikedas
Copy link
Member

ikedas commented Dec 7, 2022

Sorry, where have you submitted the Pull Request (una solicitud de incorporación de cambios)? Pull requests submitted for Sympa are listed in Pull requests page.

For more details see the documentation.

@acasadoual
Copy link
Contributor Author

I think I've already done well now.

@ikedas
Copy link
Member

ikedas commented Dec 7, 2022

Unfortunately, you deleted the branch patch-2 from which the PR created, so the PR was automatically closed. I'm very sorry, but could you please re-create the branch and submit the PR again? Please do not delete the branch once created until the PR has been merged.

@acasadoual
Copy link
Contributor Author

Sorry, I renamed my branch to Extend-info-SOAP-function

@ikedas
Copy link
Member

ikedas commented Dec 7, 2022

Thanks!

@ikedas ikedas linked a pull request Dec 7, 2022 that will close this issue
@ikedas ikedas added this to the 6.2.72 milestone Dec 8, 2022
ikedas added a commit that referenced this issue Dec 8, 2022
…tion by acasadoual & ikedas

Extend info soap function (#1542)
@ikedas
Copy link
Member

ikedas commented Dec 8, 2022

Merged. Thanks for improvement!

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

Successfully merging a pull request may close this issue.

2 participants