Skip to content

TA#77475 [14.0][ADD] partner_read_access_actions#166

Merged
abenzbiria merged 3 commits into14.0from
TA#77475
Jul 1, 2025
Merged

TA#77475 [14.0][ADD] partner_read_access_actions#166
abenzbiria merged 3 commits into14.0from
TA#77475

Conversation

@rivo2302
Copy link
Copy Markdown
Contributor

@rivo2302 rivo2302 commented Jul 1, 2025

No description provided.

@codacy-production
Copy link
Copy Markdown

codacy-production Bot commented Jul 1, 2025

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
Report missing for 1d990da1 63.64%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (1d990da) Report Missing Report Missing Report Missing
Head commit (f810d67) 3134 3018 96.30%

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#166) 11 7 63.64%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Footnotes

  1. Codacy didn't receive coverage data for the commit, or there was an error processing the received data. Check your integration for errors and validate that your coverage setup is correct.

@rivo2302 rivo2302 requested a review from abenzbiria July 1, 2025 08:57
Copy link
Copy Markdown
Contributor

@abenzbiria abenzbiria left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1- Why you are usign the list additional_actions for just one item . use instead : res.extend("action_view_sale_orde")

@rivo2302
Copy link
Copy Markdown
Contributor Author

rivo2302 commented Jul 1, 2025

@abenzbiria

My bad, I forgot to include the other actions.

In fact, there are still several actions that remain invisible — it's just that the customer only noticed the one related to sale orders.

I've now pushed the code with all the affected actions properly hidden.

@rivo2302 rivo2302 requested a review from abenzbiria July 1, 2025 15:16
@abenzbiria abenzbiria merged commit b223cb9 into 14.0 Jul 1, 2025
3 of 5 checks passed
@abenzbiria abenzbiria deleted the TA#77475 branch July 1, 2025 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants