Skip to content

Conversation

@Reyes4711-S73
Copy link
Contributor

Standard migration to 14.0

@dalonsod
Copy link
Contributor

dalonsod commented Jul 6, 2023

Hello @Reyes4711-S73 your PR is based on #337 , isn't it?

Please check, because that PR has new adjustments since you've cloned it

@Reyes4711-S73 Reyes4711-S73 force-pushed the 14.0-mig-maintenance_stock branch from edb1746 to 78728b3 Compare July 6, 2023 06:42
@Reyes4711-S73
Copy link
Contributor Author

@dalonsod I added your last changes in my migration PR

@Reyes4711-S73 Reyes4711-S73 force-pushed the 14.0-mig-maintenance_stock branch from 78728b3 to f20543b Compare July 6, 2023 07:04
@dalonsod
Copy link
Contributor

dalonsod commented Jul 6, 2023

Please check errors in tests

@Reyes4711-S73 Reyes4711-S73 force-pushed the 14.0-mig-maintenance_stock branch from f20543b to bc0a774 Compare July 7, 2023 06:22
</function>
<function model="stock.inventory" name="action_validate">
<function
eval="[[('state','=','draft'),('id', '=', ref('maintenance_stock.stock_inventory_toner'))]]"
Copy link
Member

Choose a reason for hiding this comment

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

The inventory for demo product is In Progress when installing the module:
Screen Shot 2023-07-09 at 2 00 07 PM

Perhaps it is this line that does not allow validating the inventory. Maybe the state at this moment is confirm (not tested).

Suggested change
eval="[[('state','=','draft'),('id', '=', ref('maintenance_stock.stock_inventory_toner'))]]"
eval="[[('state','=','confirm'),('id', '=', ref('maintenance_stock.stock_inventory_toner'))]]"

Can you review please ?

@dalonsod
Copy link
Contributor

@Reyes4711-S73 I've still found some issues in commit history:

  • Your migration commit has some changes that are already made in current v13 migration commit. That's because you your commit history is still created from an old version. Please check.
  • Pre-commit stuff commit has adjustments in test file that shouldn't belong to that commit. If fix in tests are needed, please place them in migration commit.

@Reyes4711-S73 Reyes4711-S73 force-pushed the 14.0-mig-maintenance_stock branch 6 times, most recently from c9e4180 to 3ee7028 Compare August 16, 2023 08:16
@Reyes4711-S73
Copy link
Contributor Author

@mamcode @dalonsod I added the merged commit and solved some issues. Please, review it

Copy link
Member

@mamcode mamcode left a comment

Choose a reason for hiding this comment

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

Hello @Reyes4711-S73 Thanks for the applied fixes.

Only a comment about the value used for field delivery_steps in stock.warehouse can you review please.

@Reyes4711-S73 Reyes4711-S73 force-pushed the 14.0-mig-maintenance_stock branch from 3ee7028 to 117ae55 Compare September 4, 2023 08:25
@Reyes4711-S73
Copy link
Contributor Author

@mamcode Change done

Copy link
Member

@mamcode mamcode left a comment

Choose a reason for hiding this comment

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

Hello @Reyes4711-S73 thanks, but there are still some smaller details to fix.

@Reyes4711-S73 Reyes4711-S73 force-pushed the 14.0-mig-maintenance_stock branch 2 times, most recently from 74ce106 to 3c453fe Compare September 18, 2023 06:43
@Reyes4711-S73
Copy link
Contributor Author

@mamcode done

Copy link
Member

@mamcode mamcode left a comment

Choose a reason for hiding this comment

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

@mamcode
Copy link
Member

mamcode commented Sep 19, 2023

@dalonsod Can you check this PR again please?

Copy link
Contributor

@dalonsod dalonsod left a comment

Choose a reason for hiding this comment

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

Tested in runboat 👍 but see at code review required changes

@Reyes4711-S73 Reyes4711-S73 force-pushed the 14.0-mig-maintenance_stock branch 2 times, most recently from 9efae1f to b573f37 Compare September 22, 2023 05:58
@mamcode
Copy link
Member

mamcode commented Sep 24, 2023

@dalonsod @Reyes4711-S73 Hello, I see errors in the Odoo test job that seem to have nothing to do with the code of this PR, how can we run the jobs again?

@Reyes4711-S73 Reyes4711-S73 force-pushed the 14.0-mig-maintenance_stock branch from b573f37 to 3ab39ac Compare September 25, 2023 05:59
@dalonsod
Copy link
Contributor

Hello @mamcode , it's a known issue, please check odoo/odoo#122569 (comment)

@Reyes4711-S73 Reyes4711-S73 force-pushed the 14.0-mig-maintenance_stock branch from 07b3541 to 22bdf26 Compare September 30, 2024 07:20
@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Oct 6, 2024
@github-actions
Copy link

github-actions bot commented Feb 9, 2025

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 Feb 9, 2025
dalonsod and others added 13 commits February 10, 2025 07:59
Currently translated at 96.7% (29 of 30 strings)

Translation: maintenance-12.0/maintenance-12.0-maintenance_stock
Translate-URL: https://translation.odoo-community.org/projects/maintenance-12-0/maintenance-12-0-maintenance_stock/es/
Currently translated at 100.0% (30 of 30 strings)

Translation: maintenance-12.0/maintenance-12.0-maintenance_stock
Translate-URL: https://translation.odoo-community.org/projects/maintenance-12-0/maintenance-12-0-maintenance_stock/pt_BR/
Currently translated at 100.0% (30 of 30 strings)

Translation: maintenance-12.0/maintenance-12.0-maintenance_stock
Translate-URL: https://translation.odoo-community.org/projects/maintenance-12-0/maintenance-12-0-maintenance_stock/it/
Currently translated at 100.0% (30 of 30 strings)

Translation: maintenance-12.0/maintenance-12.0-maintenance_stock
Translate-URL: https://translation.odoo-community.org/projects/maintenance-12-0/maintenance-12-0-maintenance_stock/it/
@Reyes4711-S73 Reyes4711-S73 force-pushed the 14.0-mig-maintenance_stock branch 2 times, most recently from c5ebbea to b72ea82 Compare February 10, 2025 07:03
@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Feb 16, 2025
@github-actions
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 Jun 22, 2025
@Reyes4711-S73 Reyes4711-S73 force-pushed the 14.0-mig-maintenance_stock branch from b72ea82 to 13863ea Compare June 23, 2025 06:06
@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jun 29, 2025
Vicent-S73 pushed a commit to Studio73/e-commerce that referenced this pull request Aug 12, 2025
@github-actions
Copy link

github-actions bot commented Nov 2, 2025

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 Nov 2, 2025
@Reyes4711-S73 Reyes4711-S73 force-pushed the 14.0-mig-maintenance_stock branch from 13863ea to 4e91dd0 Compare November 3, 2025 11:02
@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Nov 9, 2025
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.

10 participants