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

Notification not sent to the owner when a list is confirmed by a listmaster #212

Closed
ydelattre opened this issue Feb 27, 2018 · 3 comments
Closed
Labels
Milestone

Comments

@ydelattre
Copy link

Hi,

with sympa version 6.2.24, the notification is not sent to the owner when a list is confirmed by a listmaster.
Only the notification to the listmaster is sent.

We did some debug.
It appears the value of the "notify owner" checkbox is not passed to the confirm_action.tt2

From confirm action page :

<form action="/wwsympa/" method="POST">
<input name="mode" value="install" type="hidden">
--><input name="notify" value="" type="hidden"><--
  <input name="action" value="open_list" type="hidden">
  <input name="list" value="test-issue" type="hidden">
  <input name="previous_action" value="set_pending_list_request" type="hidden">
  <div>
    <input class="MainMenuLinks" id="response_action_confirm" name="response_action_confirm" value="Confirmer" type="submit">
    <input class="MainMenuLinks" id="response_action_cancel" name="response_action_cancel" value="Retour" type="submit">
  </div>
</form>

The value is correct in the page's post parameters :

action : open_list
list : test-issue
mode : install
notify : on
previous_action : set_pending_list_request
serial : 1

Thanks for your help.

@ikedas ikedas added the bug label Feb 27, 2018
ikedas added a commit to ikedas/sympa that referenced this issue Feb 28, 2018
@ikedas
Copy link
Member

ikedas commented Feb 28, 2018

Hi @ydelattre,

Can you apply this patch and check if problem you reported will be solved? Thanks.

@ydelattre
Copy link
Author

Hi @ikedas,

this patch solve the issue :)
Thanks a lot for your reactivity !

ikedas added a commit that referenced this issue Feb 28, 2018
Issue #212: Notification not sent to the owner when a list is confirmed by a listmaster
@ikedas ikedas added this to the 6.2.26 milestone Feb 28, 2018
@ikedas
Copy link
Member

ikedas commented Feb 28, 2018

Thanks for confirmation.

The patch was applied. This issue is closed.

@ikedas ikedas closed this as completed Feb 28, 2018
mpkut pushed a commit to mpkut/sympa that referenced this issue Mar 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants