-
-
Notifications
You must be signed in to change notification settings - Fork 304
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
[13.0] multi company improvements #374
[13.0] multi company improvements #374
Conversation
Hi @sbidoul! Thank you very much for this contribution. As the addon you are improving does not have a declared maintainer, I take the opportunity to mention that you can consider adopting it. To do so, please read the maintainer role description, and, if interested, create a pull request to add your GitHub login to the |
Looks like this bot message about maintainers is not working as exepected, as I am a maintainer... |
3b009a4
to
3175dbf
Compare
- The mis.report.instance should allow to define as many companies as the user is allowed to operate on when logged. - The company is only a required field if the flag 'multi_company' is not set.
…tor and fix tests
It is the intersection of selected companies on the report and env.companies (which apply as multi company record rules) Hide the company_id field in multi_company reports.
3175dbf
to
d039e4e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your work!
@MiquelRForgeFlow @JordiBForgeFlow have you had a chance to look at this followup to your previous PR ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey, sorry for the long delay here... I have tested and the changes are added are perfectly fine with me. Thanks 👍
This is #327 with the the few improvements I suggested in review comments.
@MiquelRForgeFlow @JordiBForgeFlow let me know what you think.
closes #379
closes #327