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

[16.0][MIG] hotel #198

Merged
merged 143 commits into from
Sep 28, 2024
Merged

[16.0][MIG] hotel #198

merged 143 commits into from
Sep 28, 2024

Conversation

sachin-serpentcs
Copy link

No description provided.

Atul Makwana(SerpentCS) and others added 30 commits September 29, 2023 18:04
francesco-ooops and others added 11 commits September 29, 2023 18:04
Currently translated at 100.0% (520 of 520 strings)

Translation: vertical-hotel-15.0/vertical-hotel-15.0-hotel
Translate-URL: https://translation.odoo-community.org/projects/vertical-hotel-15-0/vertical-hotel-15-0-hotel/it/
Currently translated at 100.0% (520 of 520 strings)

Translation: vertical-hotel-15.0/vertical-hotel-15.0-hotel
Translate-URL: https://translation.odoo-community.org/projects/vertical-hotel-15-0/vertical-hotel-15-0-hotel/it/
Currently translated at 100.0% (520 of 520 strings)

Translation: vertical-hotel-15.0/vertical-hotel-15.0-hotel
Translate-URL: https://translation.odoo-community.org/projects/vertical-hotel-15-0/vertical-hotel-15-0-hotel/it/
Currently translated at 100.0% (520 of 520 strings)

Translation: vertical-hotel-15.0/vertical-hotel-15.0-hotel
Translate-URL: https://translation.odoo-community.org/projects/vertical-hotel-15-0/vertical-hotel-15-0-hotel/it/
Currently translated at 100.0% (520 of 520 strings)

Translation: vertical-hotel-15.0/vertical-hotel-15.0-hotel
Translate-URL: https://translation.odoo-community.org/projects/vertical-hotel-15-0/vertical-hotel-15-0-hotel/it/
Currently translated at 100.0% (520 of 520 strings)

Translation: vertical-hotel-15.0/vertical-hotel-15.0-hotel
Translate-URL: https://translation.odoo-community.org/projects/vertical-hotel-15-0/vertical-hotel-15-0-hotel/it/
Currently translated at 100.0% (520 of 520 strings)

Translation: vertical-hotel-15.0/vertical-hotel-15.0-hotel
Translate-URL: https://translation.odoo-community.org/projects/vertical-hotel-15-0/vertical-hotel-15-0-hotel/it/
Currently translated at 100.0% (520 of 520 strings)

Translation: vertical-hotel-15.0/vertical-hotel-15.0-hotel
Translate-URL: https://translation.odoo-community.org/projects/vertical-hotel-15-0/vertical-hotel-15-0-hotel/it/
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jan 28, 2024
@github-actions github-actions bot closed this Mar 3, 2024
@JayVora-SerpentCS JayVora-SerpentCS removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jun 27, 2024
@@ -8,7 +8,7 @@
Services Pvt. Ltd., OpenERP SA",
"category": "Hotel Management",
"website": "https://github.com/OCA/vertical-hotel",
"depends": ["sale_stock", "account"],
"depends": ["sale_stock", "account", "uom"],
Copy link
Member

Choose a reason for hiding this comment

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

@pavan-serpentcs Any good reason to add "uom" in dependency?

The stock modules already have dependency products, and the products have the uom module as a dependency.


_inherit = "account.move"

@api.model

Choose a reason for hiding this comment

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

Suggested change
@api.model
@api.model_create_multi

Copy link
Member

@nikul-serpentcs nikul-serpentcs left a comment

Choose a reason for hiding this comment

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

Code Review LGTM

@OCA-git-bot
Copy link
Contributor

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

Copy link
Member

@murtuzasaleh murtuzasaleh left a comment

Choose a reason for hiding this comment

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

LGTM

@JayVora-SerpentCS JayVora-SerpentCS merged commit 2f97245 into OCA:16.0 Sep 28, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.