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

New hide external and non employee users in combobox #402

Merged
merged 2 commits into from
Nov 27, 2023

Conversation

atm-thibaultf
Copy link

No description provided.

@john-botella
Copy link
Member

C'est un backport depuis Dolibarr ?

@atm-thibaultf
Copy link
Author

atm-thibaultf commented Sep 11, 2023

@atm-john

Yes, j'avais fait cette PR chez nous validée par Léna : #401

Mais en parallèle j'avais fait cette PR en standard et Eldy m'avait fait des retours : Dolibarr#25740

Du coup j'ai corrigé ma PR standard et j'ai donc refait une PR chez nous pour être raccord, mais je ne sais pas si c'est bien la bonne pratique ?!

@john-botella john-botella merged commit 4c47570 into 17.0_atm Nov 27, 2023
@john-botella john-botella deleted the NEW_hide_external_user_in_combobox branch November 27, 2023 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants