Skip to content

[REF] mis_builder: create dedicated 'MIS Reports' Entry menu for mis_builder application#732

Draft
legalsylvain wants to merge 2 commits into
OCA:18.0from
grap:18.0-REF-mis_builder-add-main-menu-entries
Draft

[REF] mis_builder: create dedicated 'MIS Reports' Entry menu for mis_builder application#732
legalsylvain wants to merge 2 commits into
OCA:18.0from
grap:18.0-REF-mis_builder-add-main-menu-entries

Conversation

@legalsylvain

@legalsylvain legalsylvain commented Sep 26, 2025

Copy link
Copy Markdown
Contributor
  1. Create dedicated menu entry

  2. change icon.

Icon before

image

Icon after

(put some similar style as in V18).

image

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @sbidoul,
some modules you are maintaining are being modified, check this out!

@legalsylvain

Copy link
Copy Markdown
Contributor Author

Hi @sbidoul

i have a question for the groups.

for the time being :

  • menu of "MIS Reporting" is accessible to account.group_account_manager
  • menu of "MIS Configuration" is accessible to account.group_account_manager
  • all models are readable by base.group_user
  • all models are writable by account.group_account_manager

As a result, by API, any user can read all the data, but it is hidden in the UI, and only visible by accountant.

What I propose :

  • creation of a mis_builder.user group, that can read report. (but not templates).
  • create of a mis_builder.manager group, that can edit all MIS models
    In the migration script, all the users of account.group_account_manager are now member of account.group_account_manager to avoid loosing access during migration.

Is that what you had in mind?

Thanks !

@legalsylvain

Copy link
Copy Markdown
Contributor Author

Hi @OCA/mis-builder-maintainers ! friendly reminder here.

Thanks !

@legalsylvain

legalsylvain commented Oct 28, 2025

Copy link
Copy Markdown
Contributor Author

Hi @OCA/mis-builder-maintainers. any point of view, regarding my question ?

#732 (comment)

thanks !

@legalsylvain legalsylvain force-pushed the 18.0-REF-mis_builder-add-main-menu-entries branch from 92f1a5a to ecd7172 Compare October 28, 2025 14:05
@legalsylvain legalsylvain force-pushed the 18.0-REF-mis_builder-add-main-menu-entries branch from ecd7172 to 07113cf Compare November 12, 2025 21:04
@github-actions

Copy link
Copy Markdown

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions Bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Mar 15, 2026
@legalsylvain

Copy link
Copy Markdown
Contributor Author

@sbidoul : any chance to have an answer on that topic ? Or I close this PR ?

(if it's more simple, we can have a little talk, directly).

thanks !

@sbidoul sbidoul removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Mar 19, 2026
@sbidoul

sbidoul commented Mar 19, 2026

Copy link
Copy Markdown
Member

Hi Sylvain,

To progress in small steps, maybe we can start with the security groups first without changing the menus?

There was some preliminary discussion about this in #639 (comment). So perhaps we could create a new issue to discuss a rework of the groups?

@legalsylvain

Copy link
Copy Markdown
Contributor Author

To progress in small steps, maybe we can start with the security groups first without changing the menus?

Of course ! I'll take a look on the previous discussion and propose a PR if possible !

@sbidoul

sbidoul commented Mar 19, 2026

Copy link
Copy Markdown
Member

I suggest making an issue with a proposal first, but up to you if you prefer expressing the proposal in code :)

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