search

OCA / vertical-rental

GNU Affero General Public License v3.0
19 stars 31 forks source link

[IMP] add data product uom week. #31

Open yweng8111 opened 1 year ago

marylla commented 1 year ago

@yweng8111 Can you please improve the test coverage?

marylla commented 1 year ago

@yweng8111 Do you know what's the problem of the pre-commit? How can we fix it?

Version 3.9.7 was not found in the local cache
Error: Version 3.9.7 with arch x[6](https://github.com/OCA/vertical-rental/actions/runs/3702369286/jobs/6272568713#step:3:7)4 not found
The list of all available versions can be found here: https://raw.githubusercontent.com/actions/python-versions/main/versions-manifest.json
marylla commented 9 months ago

@yweng8111 Can you check this please?

olaf-wagner commented 7 months ago

Can we just update the python version in the pre-commit workflow from 3.9.7 to 3.9.18 please?

marylla commented 7 months ago

@chandni299 Please remind me: What is this useful for? I still can just create rental orders with uom hours, days or months - but not weeks. It's a while ago that I tested rental orders... ^^

chandni299 commented 6 months ago

Hello @marylla I think this is the base code for product uom week, the actual rental order having weeks will be created when we have this PR-32. Let me correct if I am wrong @yweng8111