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

[17.0][UPG] viin_brand_base_import: upgrade to version 17.0 #514

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

Conversation

SonCrits
Copy link
Contributor

@SonCrits SonCrits commented Jul 23, 2024

viin_brand_base_import

PR này để:

  • Nâng cấp mô đun viin_brand_base_import lên phiên bản 17.0
  • Chỉ thay đổi về tên file giữa 16 và 17 của odoo. ngoài ra không có thêm danh sách cần xpath

Todo

  • 2 help sau ( trên js ) chưa tìm được chỗ odoo sử dụng ở đâu để có thể xác định trước khi code
  • Use YYYY to represent the year, MM for the month and DD for the day. Include separators such as a dot, forward slash or dash. You can use a custom format in addition to the suggestions provided. Leave empty to let Odoo guess the format (recommended)
  • Use HH for hours in a 24h system, use II in conjonction with 'p' for a 12h system. You can use a custom format in addition to the suggestions provided. Leave empty to let Odoo guess the format (recommended)
brand_base_import.mp4

@viinbot viinbot added the 🚀 Queue PR/Commit is added in build queue label Jul 23, 2024
@SonCrits SonCrits force-pushed the 17_viin_brand_base_import branch from 5353b12 to e16fa97 Compare July 23, 2024 04:55
@viinbot viinbot added 🚀 Building PR/Commit is start building all step in script 🚀 Running PR/Commit is done for everything and ready for manually test and removed 🚀 Queue PR/Commit is added in build queue 🚀 Building PR/Commit is start building all step in script labels Jul 23, 2024
@viinbot
Copy link
Contributor

viinbot commented Jul 23, 2024

@SonCrits Viindoo Test Suite has passed!

@SonCrits SonCrits requested a review from royle-vietnam July 23, 2024 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 Running PR/Commit is done for everything and ready for manually test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants