Closed nfmohit closed 2 months ago
Build files for e08e57380c4103668a395b81d3f575208b1d1e5a have been deleted.
Hey @nfmohit, the code looks good with regard to the IB, but I have a question as to whether the AC as actually correct here. Please see my comment on the issue: #9150 (comment)
Responded, thank you @techanvil !
Summary
Addresses issue:
9150
Relevant technical choices
This PR improves the RRM entity access checks by ensuring that the saved publication exists in the current user's publisher center.
Unrelated changes
Reader_Revenue_Manager
was missing test coverage forModule_With_Owner
andModule_With_Scopes
traits. I've added them.PR Author Checklist
Do not alter or remove anything below. The following sections will be managed by moderators only.
Code Reviewer Checklist
Merge Reviewer Checklist