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

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). 🤖

/ocabot merge patch

On my way to merge this fine PR!
Prepared branch 19.0-ocabot-merge-pr-608-by-lmignon-bump-patch, awaiting test results.

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

I have some worries about this. I had not noticed this feature before.

Is it not problematic that merely installing the fastapi module changes the way Odoo determines the database globally? Is it not preferable to do this in a standalone module that would be independent of fastapi?

I have some worries about this. I had not noticed this feature before.

Is it not problematic that merely installing the fastapi module changes the way Odoo determines the database globally? Is it not preferable to do this in a standalone module that would be independent of fastapi?

Hi @sbidoul

Odoo vanilla accepts db query param, so this is trying to make the same behavior happen with fastapi. I haven't seen the need to patch it in any other place.

Without db param odoo redirects you to the database selector. Example in a repo without this patch: http://oca-ddmrp-18-0-0cd23739c209.runboat.odoo-community.org/web/

Whereas if you include the db param you get redirected to that specific database: http://oca-ddmrp-18-0-0cd23739c209.runboat.odoo-community.org/odoo?db=b2a480f47-259f-4bdd-9d4c-dcb5194260ed-baseonly

What do you think?