Skip to content

Conversation

@fw-bot-adhoc
Copy link
Contributor

[FIX] portal_backend: add missing company fields in session_info

Portal users with multi-company access got JS errors when switching companies
due to missing child_ids, parent_id and disallowed_ancestor_ companies fields in session_info.
Align implementation with Odoo core.

[FIX] portal_holidays: add ir.rule for multi-company hr.leave.type access

Portal backend users with multi-company access were getting access errors
when trying to access the Holidays app due to hr.leave.type having a global multi-company rule
that restricts access based on company_ids.

Added specific ir.rule for group_portal_backend_holiday allowing access to time off types
without company or belonging to user's companies.

Forward-Port-Of: #314

stdout:
Auto-merging portal_backend/models/ir_http.py
CONFLICT (content): Merge conflict in portal_backend/models/ir_http.py

stderr:
@roboadhoc
Copy link

Pull request status dashboard

@fw-bot-adhoc
Copy link
Contributor Author

@ced-adhoc @jcadhoc cherrypicking of pull request #314 failed.

stdout:

Auto-merging portal_backend/models/ir_http.py
CONFLICT (content): Merge conflict in portal_backend/models/ir_http.py

Either perform the forward-port manually (and push to this branch, proceeding as usual) or close this PR (maybe?).

In the former case, you may want to edit this PR message as well.

⚠️ after resolving this conflict, you will need to merge it via @roboadhoc.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@fw-bot-adhoc
Copy link
Contributor Author

@ced-adhoc @jcadhoc this forward port of #314 is awaiting action (not merged or closed).

2 similar comments
@fw-bot-adhoc
Copy link
Contributor Author

@ced-adhoc @jcadhoc this forward port of #314 is awaiting action (not merged or closed).

@fw-bot-adhoc
Copy link
Contributor Author

@ced-adhoc @jcadhoc this forward port of #314 is awaiting action (not merged or closed).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants