[16.0] [MIG] document_hosting

victor-champonnois commented 8 months ago

Description

Migrating to 16
fix settings view and put it in it's own section
add roadmap

Odoo task (if applicable)

https://gestion.coopiteasy.be/web#id=11971&action=475&active_id=492&model=project.task&view_type=form&menu_id=

Checklist before approval

[ ] Tests are present (or not needed).
[ ] Credits/copyright have been changed correctly.
[ ] Change log snippet is present.
[ ] (If a new module) Moving this to OCA has been considered.

polchampion commented 8 months ago

@victor-champonnois Can you put it in test?

victor-champonnois commented 8 months ago

@polchampion done

victor-champonnois commented 8 months ago

@polchampion thanks for the test, it's fixed. I've also fixed the display of category tree (migrating to bootstrap v5)

victor-champonnois commented 8 months ago

@polchampion ...and it's just deployed to test

polchampion commented 8 months ago

functional test on 16-test-divers

[x] internal user can create, upload, share and make documents public
[x] public user can display and download public documents
[ ] portal user can display and download published documents NOK, only sees the public document

@victor-champonnois Only public documents are displayed in Documents, whereas all published documents should show up for portal and internal users, right?

victor-champonnois commented 8 months ago

@polchampion to me it's working. As a portal user I see the doc "test partage doc interne" that is published but not public. It's in the "categorie document de test though", so you need to click on the category to see it.

polchampion commented 8 months ago

functional test on 16-test-divers

[x] internal user can create, upload, share and make documents public
[x] public user can display and download public documents
[x] portal user can display and download published documents

@victor-champonnois Oh I had totally missed that, thank you. I'll suggest a change to better display public documents as a category, but as far as the porting goes, it all looks good to me

github-grap-bot commented 8 months ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

github-grap-bot commented 8 months ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

victor-champonnois commented 4 months ago

/ocabot merge nobump

github-grap-bot commented 4 months ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-310-by-victor-champonnois-bump-nobump, awaiting test results.

github-grap-bot commented 4 months ago

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

coopiteasy / addons