[19.0][IMP] spreadsheet_dashboard_oca: manage dashboard share links - #130
[19.0][IMP] spreadsheet_dashboard_oca: manage dashboard share links#130volkantasci wants to merge 1 commit into
Conversation
c494e67 to
62beb16
Compare
|
Hi @pedrobaeza and @victoralmau 👋 Just a gentle check-in — if anyone has a moment, this PR would really benefit from a look whenever you're available. All CI checks are green (pre-commit, Odoo/OCB tests, codecov, runboat). No pressure at all — happy to make any changes you suggest. Thanks for everything you do! |
|
I think this should be part of the main module, as the share option is something already in it. |
|
Hi @pedrobaeza, thanks for the feedback — and apologies for the slow reply. Agreed, the share-link management UI belongs in the main module. To clarify what this PR adds: core's You're right that a separate companion module isn't the right shape for a small, dependency-free addition like this. We'll rework the PR to fold the feature directly into |
Fold the dashboard share management feature previously proposed as the standalone spreadsheet_dashboard_share_oca module into the main module, following maintainer feedback on the original PR. Core's spreadsheet_dashboard only lets users create share links; there is no UI to list or revoke them. This adds a "Manage shares" dialog (list / copy / revoke) and a share-count badge next to the dashboard name, plus an ir.rule letting dashboard managers manage the shares of every user. Python and JS tests are moved along and the module version is bumped to 19.0.2.0.0.
62beb16 to
2b583a2
Compare
|
Reworked as discussed, @pedrobaeza 👋 The PR no longer adds a standalone module — the share management is folded into
No new dependency is introduced. When you have a moment, could you take another look? Happy to adjust anything. |
Following maintainer feedback, the share-link management feature previously proposed as the standalone
spreadsheet_dashboard_share_ocamodule is now folded directly into the mainspreadsheet_dashboard_ocamodule.Core's
spreadsheet_dashboardonly lets users create share links; there is no UI to list or revoke them. This PR adds:ir.rulesospreadsheet_dashboard.group_dashboard_managerusers can manage the shares created by any user (other users manage their own).Python and JS tests moved along; module version bumped to
19.0.2.0.0.