Migration to version 17.0

[OCA-git-bot]

Todo

https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-17.0

Modules to migrate

• dms - By @tva-subteno-it - [17.0][MIG] DMS : Migration to 17.0 #323
• dms_attachment_link - By @victoralmau - [17.0][MIG] dms_attachment_link: Migration to 17.0 #373
• dms_storage - By @BernatObrador - [17.0][MIG] dms_storage: Migration to 17.0 #362

Missing module? Check https://github.com/OCA/maintainer-tools/wiki/%5BFAQ%5D-Missing-modules-in-migration-issue-list

[tva-subteno-it]

Hi, I'm currently migrating dms to v17, and I noticed that the web_drop_target module was removed from the manifest in v16, but the many_drop_target.js file using it was kept. From what I read in the discussion of the migration issue to v16 this module is no longer needed ? Can I then delete many_drop_target.js ? I don't understand how it can works if web_drop_target is not installed and is not useful anymore.

[tva-subteno-it]

Also I noticed in src/xml a file named views.xml. It extends from DocumentViewer but it seems to have been removed since Odoo v16.
You just removed the file from the manifest during the v15->v16 migration of DMS, but you didn't deleted the file itself, is there a reason ?

[pedrobaeza]

@vancouver29 can you answer?

[tva-subteno-it]

I have the answer for DocumentViewer. It's not useful at all since now displaying an image/pdf is handled by the useFileViewer hook, which displays the web.FileViewer component.

[pedrobaeza]

Great, can you propose the PR?

[tva-subteno-it]

Sure thing, it's the PR #323 currently in draft since I'm still working on it.

[tva-subteno-it]

Icons has drastically changed in V17, so I changed the icon in order to fit with the global design. I'm no designer so it might not be perfect. What do you think about it ? I'm not sure if the curved style of the pinkish part is right, maybe a more straight cut could be better.

But I think overall it fits quite well among the other icons. I don't know if I'm allowed to use the icon of the enterprise version (I don't think I am).

[tva-subteno-it]

About the web_drop_target and many_drop_target.js, indeed it's now in Odoo. No need to use other modules. At least, for DMS it works fine without it, you can still drop one or more files into the views.

[frankofno]

is this v17 working with odoo ce v 17?

the official OCA app page is listing dms only up to odoo v16
https://odoo-community.org/shop/document-management-system-5619?search=dms#attr=940531

[tva-subteno-it]

Hi @frankofno , yes it is working for ce v17, that's what I have been using. The module is not listed yet because the migration is still pending. I have finished my migration work, and it's now up to the reviewer to merge or not the PR. But if now how to do it, you can still download it manually from my fork: https://github.com/tva-subteno-it/dms/tree/17.0-mig-dms

[rishi-manglesh]

Hi @tva-subteno-it,

I am also following this migration to use this app in my database, so would like to thank you for the superb migration work.
However, while testing I could not Create Directory. Whatever I try, I get permission issues while creating the root directory. I have Document Manager role and also assigning these permissions in groups tab of the directory creation form. Am I missing something or is it a bug that user does not inherit the permissions.