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

PUT /controls/stakeholder/{id} does not correctly save "stakeholderGroups" field #66

Closed
carlosthe19916 opened this issue Mar 24, 2021 · 1 comment · Fixed by #129
Closed

Comments

@carlosthe19916
Copy link
Contributor

How to reproduce:

  • Create a stakeholder and assign him at least one group. This can be done using POST /controls/stakeholder.
  • Edit the previously created stakeholder and change the set of groups assigned before. This can be done using PUT /controls/stakeholder/{id}
  • The new set of groups assigned to the stakeholder adds the new groups but doesn't delete the previous one.

Here is an example using the UI. In the following video we see a stakeholder with 3 groups and then, using the edit modal, I tried to have only 1 group instead of 3; see that even though the PUT call is correct the data is not updated in the backend.

Peek.2021-03-24.11-10.mp4
@mrizzi
Copy link
Contributor

mrizzi commented Mar 24, 2021

Thanks @carlosthe19916 for reporting this.
I think it's an effect of the same issue we discussed in the last days and already reported in #64 that prevents stakeholder PUT endpoint to work as expected.

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