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

[lcc_members] make team_id field mandatory in res.partner form view #171

Closed
MatSarr opened this issue Mar 11, 2022 · 4 comments · Fixed by #165, #201 or #207
Closed

[lcc_members] make team_id field mandatory in res.partner form view #171

MatSarr opened this issue Mar 11, 2022 · 4 comments · Fixed by #165, #201 or #207
Assignees
Labels

Comments

@MatSarr
Copy link

MatSarr commented Mar 11, 2022

No description provided.

@stephansainleger
Copy link
Collaborator

Solution must be changed: it is not a good idea to make the field mandatory in the res.partner form view, as lot of partners can have no team_id.
The importance of having a local group is for membership only, then the new solution proposed is to :

  • set the team_id field optional again in res.partner form view
  • add a team_id mandatory field in the membership registration wizard: if a membership is registered, we have to set a team-id
  • ensure that the team_id field is filled if the free_member field is checked

@stephansainleger
Copy link
Collaborator

It is urgent to remove the mandatory attribute of this field, as it prevents any change in the public or position profiles in which the field is not displayed...

@MatSarr
Copy link
Author

MatSarr commented May 9, 2022

Even if local group is field, it's empty when a membership is registered.
I'm not satisfied with that.

@stephansainleger
Copy link
Collaborator

Fix delivered 42f10d0

@stephansainleger stephansainleger moved this from In Progress to To Review in Lokavaluto DEV Team work May 10, 2022
Repository owner moved this from To Review to Done in Lokavaluto DEV Team work May 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment