Skip to content

Comments

[18.0][IMP]: spreadsheet_dashboard_oca: Add to dashboard feature from spreadsheet#83

Merged
OCA-git-bot merged 1 commit intoOCA:18.0from
chrisandrewmann:18.0
Oct 27, 2025
Merged

[18.0][IMP]: spreadsheet_dashboard_oca: Add to dashboard feature from spreadsheet#83
OCA-git-bot merged 1 commit intoOCA:18.0from
chrisandrewmann:18.0

Conversation

@chrisandrewmann
Copy link
Contributor

@chrisandrewmann chrisandrewmann commented Oct 20, 2025

New feature to easily add an existing spreadsheet to a dashboard, via the File menu.

image

Usage:

  • User should be part of group "spreadsheet_dashboard.group_dashboard_manager" (Dashboard Admin) or "base.group_system" (System Admin)

Ping: @CarlosRoca13 @etobella

Copy link
Member

@flotho flotho left a comment

Choose a reason for hiding this comment

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

Hi,
Thanks
Successfully Tested on runboat,

Additional question, what is the process to delete the entry from dashboard ?
Regards

@chrisandrewmann
Copy link
Contributor Author

chrisandrewmann commented Oct 25, 2025

Hi, Thanks Successfully Tested on runboat,

Additional question, what is the process to delete the entry from dashboard ? Regards

Hi @flotho
This was a problem I ran into as normally the Administration/Settings (base.group_system) users are able to go to:

  • Dashboards
  • Configuration > Dashboards
  • Select a group section, then click the bin to delete a board from there

However the "Dashboard / Admin" group, whilst allowing creation of dashboards and groups does NOT allow access to ir.model.data so for these users they will see this error if trying to access any dashboard group in configuration menu:
image

I didn't want to change this as it seems standard Odoo behaviour, so on my own personal system, i've just added a manual additional Access Right permission to the Dashboard Admin group as follows:
image

This means that all "Dashboard Admin" users can then both create AND delete dashboards, but it strikes me as a possible security flaw and maybe a bug that should be reported to Odoo?
Hence why I decided not to add the permission in the module. What do you think?

@pedrobaeza pedrobaeza added this to the 18.0 milestone Oct 27, 2025
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Tested and working, thanks.

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-83-by-pedrobaeza-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit d977561 into OCA:18.0 Oct 27, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at e8c140d. Thanks a lot for contributing to OCA. ❤️

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.

4 participants