OpusVL / docker-odoo-custom

An Odoo docker image with extended capabilities and fixes over the official one
Other
1 stars 1 forks source link

11.0 fix filestore perms #15

Open PeterAlabaster opened 5 years ago

PeterAlabaster commented 5 years ago

Fixes #14 for 11.0, not for 12.0 (that will need a seperate PR)

Nick-OpusVL commented 5 years ago

This for the new car dealership project?

PeterAlabaster commented 5 years ago

This for the new car dealership project?

Yes, however it's beneficial for any Odoo 11 which is based on docker-odoo-custom

Nick-OpusVL commented 5 years ago

Oh @PeterAlabaster I also suggest adding to the README about the change, to say that users might have to explicitly use -u root or -u odoo in their docker command line as appropriate when running non-odoo commands whereas before they might not have had to.

Nick-OpusVL commented 5 years ago

Perhaps we should be versioning these so breaking changes like this have a version number bump at the third level (e.g. 11.0.2.7 -> 11.0.3.0) and non-breaking changes at the fourth (e.g. 11.0.3.0 -> 11.0.3.1), with the third-level having a branch (11.0.3) and fourth-level releases being a tag (11.0.3.0). This would allow us to pin projects using a particular 'compatibility version' into that series until we can get around to reviewing its usage and upgrading it.

We could freeze the two-level ones where they currently are and use three-level or four-level docker tags in our FROM clauses from this point on.