Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[IMP] add data product uom week. #31

Open
wants to merge 1 commit into
base: 12.0
Choose a base branch
from

Conversation

yweng8111
Copy link
Contributor

No description provided.

@marylla
Copy link

marylla commented Dec 20, 2022

@yweng8111 Can you please improve the test coverage?

@marylla
Copy link

marylla commented Dec 20, 2022

@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
Copy link

marylla commented Dec 8, 2023

@yweng8111 Can you check this please?

@olaf-wagner
Copy link
Contributor

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

@yweng8111 yweng8111 force-pushed the 12.0-rental_base_uom_week branch from 5cf74bf to b0252a4 Compare February 1, 2024 16:32
@yweng8111 yweng8111 force-pushed the 12.0-rental_base_uom_week branch from b0252a4 to 03c0bd7 Compare February 5, 2024 15:18
Copy link
Contributor

@chandni299 chandni299 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good!

@marylla
Copy link

marylla commented Feb 16, 2024

@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
Copy link
Contributor

chandni299 commented Feb 27, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants