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

Migration to version 9.0 #100

Closed
6 of 37 tasks
pedrobaeza opened this issue Oct 14, 2015 · 23 comments
Closed
6 of 37 tasks

Migration to version 9.0 #100

pedrobaeza opened this issue Oct 14, 2015 · 23 comments
Labels
help wanted stale PR/Issue without recent activity, it'll be soon closed automatically. work in progress
Milestone

Comments

@pedrobaeza
Copy link
Member

pedrobaeza commented Oct 14, 2015

Todo

https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-9.0

Modules to migrate

@SodexisTeam
Copy link
Member

For info, we are porting product_m2mcategories module

@pedrobaeza
Copy link
Member Author

Thanks for the great work migrating modules! What do you think of renaming the module to product_multi_category to follow naming conventions and avoid plurals?

@atchuthan
Copy link
Member

yes, we will rename it to product_multi_category.

Also, with regards to product_images module, it seems there is a pending migration at #57 in which it is renamed to product_multi_image.

Should I start the porting of product_multi_image module based on this PR?

@pedrobaeza
Copy link
Member Author

@atchuthan, the correct module to be migrated to 9.0 is #135, that is the work that I started in the PR you have mentioned, but with a better base. The heavy load of the module has been moved to a generic module (base_multi_image) to be reusable, that is already merged in https://github.com/OCA/server-tools/tree/8.0/base_multi_image, so you'll need to also migrate that one, but you will see that you have to do very little, and it's by far more powerful than the previous versions.

@atchuthan
Copy link
Member

okay, thanks for the info.

@SodexisTeam
Copy link
Member

For info, we are migrating product_sequence.

There seems to be a PR associated with this module but no activity on it(since 10 Nov 2015).
So, we are going to create a new PR.

@leemannd
Copy link
Contributor

leemannd commented May 27, 2016

Doing the product_dimension in #159

@flotho
Copy link
Member

flotho commented Jun 23, 2016

Hi,

You could add product dimension : #166

@flotho
Copy link
Member

flotho commented Jun 23, 2016

Please also consider this one #167 as WIP

@pedrobaeza
Copy link
Member Author

Why 2 times product_dimension module?

@flotho
Copy link
Member

flotho commented Jul 5, 2016

Hi @pedrobaeza :

@flotho
Copy link
Member

flotho commented Jul 5, 2016

@pedrobaeza , please consider #171 as migration of product_custom_attributes only ;-)

@flotho
Copy link
Member

flotho commented Jul 5, 2016

hi @pedrobaeza , please consider #172 as the migration of product_categ_only ;-)

@yajo
Copy link
Member

yajo commented Aug 3, 2016

Starting with product_custom_info

@flotho
Copy link
Member

flotho commented Aug 24, 2016

@yajo , @pedrobaeza , just to be sure, it looks like product_custom_info is a replacement of product_custom_attributes, so, what about product_categ_attributes ? will it be a replacement too or any other recommandation ?

regards

@yajo
Copy link
Member

yajo commented Aug 24, 2016

Well, we actually do not use any of both, but given OCA/server-tools#492 provides a great framework for custom information into any model, I think that should be the path. I guess I'm a little biased being the main developer of all the custom_info collection, but I think other modules that need migration should have 1 version (maybe v9) of deprecation (just depend on its *_custom_info counterpart and add a migration script that moves old to new).

However, as long as modules do not interfere with each other, there should be no problem on having all of them if you need them.

@pedrobaeza
Copy link
Member Author

No, @yajo, OCA's rules forbid to have 2 modules for the same purpose.

@flotho, product_categ_attributes should be adapted to the new system, but it's very easy (a field custom_info_template_id in product.category + onchange on product.product when changing the category for bringing the custom info template).

@flotho
Copy link
Member

flotho commented Aug 24, 2016

Hi @pedrobaeza thanks for your answer,
so if I understand well, using the cutom info modules instead of the custom attributes one + a migration for the product_categ_attributes will do the job. am I right?

@pedrobaeza
Copy link
Member Author

Yes, you are

@yajo
Copy link
Member

yajo commented Jun 9, 2017

#177 was merged

@fanha99
Copy link

fanha99 commented Jan 2, 2018

there are 2 line of product_custom_info in the list

@pedrobaeza
Copy link
Member Author

Extra one removed.

@pedrobaeza pedrobaeza reopened this Jan 2, 2018
@github-actions
Copy link

There hasn't been any activity on this issue in the past 6 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 issue 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 Oct 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted stale PR/Issue without recent activity, it'll be soon closed automatically. work in progress
Projects
None yet
Development

No branches or pull requests

7 participants