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 11.0 #1008

Open
pedrobaeza opened this issue Oct 3, 2017 · 50 comments

Comments

Projects
None yet
@pedrobaeza
Copy link
Member

commented Oct 3, 2017

Todo

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

Repository split

As talked in this mailing list thread this repository is very crowded, so for 11.0, an split has been done across several repositories. Please check current spreadsheet for the initial distribution plan. If there's a module not included here or that you have doubts, please ask in this issue:

https://docs.google.com/spreadsheets/d/1Xg95cW4TFMf_Lo5i_CZC_qOOfN8RgxPRc0LJTLTkdUI/edit?usp=sharing

Modules to migrate

@novawish

This comment has been minimized.

Copy link

commented Nov 12, 2017

Hello @pedrobaeza,
I did not see (in here) which repository that module base_user_role should belonged to.

Should I create a new PR to server-backend as you mentioned in #1036 or I just keep the PR like that?

@pedrobaeza

This comment has been minimized.

Copy link
Member Author

commented Nov 22, 2017

That's because that module doesn't exist when the list was created. My opinion is that it should go to server-ux

@simahawk

This comment has been minimized.

Copy link
Contributor

commented Nov 29, 2017

mail_environment @simahawk #1074

@zamberjo

This comment has been minimized.

Copy link
Member

commented Nov 29, 2017

@pedrobaeza auth_brute_force is not included

@pedrobaeza

This comment has been minimized.

Copy link
Member Author

commented Nov 29, 2017

@simahawk please don't include - [ ] in your comment, as it adds another task to do in the issue summary link, which is not correct, as there's already a bullet in the main comment.

@zamberjo it was in the list... I have included a link to your PR there.

@pedrobaeza

This comment has been minimized.

Copy link
Member Author

commented Nov 29, 2017

@zamberjo or are you talking about the split list? I'm thinking that the module should go to server-auth.

@njeudy

This comment has been minimized.

Copy link

commented Dec 3, 2017

sentry @njeudy #1082

@nadiaafa

This comment has been minimized.

Copy link

commented Dec 21, 2017

@pedrobaeza i will work on auth_session_timeout

@pedrobaeza

This comment has been minimized.

Copy link
Member Author

commented Dec 21, 2017

@nadiaafa that module should go to server-auth

@jlzhou

This comment has been minimized.

Copy link

commented Dec 27, 2017

base_external_dbsource
base_external_dbsource_mssql
base_external_dbsource_mysql
base_external_dbsource_odbc
base_external_dbsource_oracle
base_external_dbsource_sqlite

#1107

@pedrobaeza

This comment has been minimized.

Copy link
Member Author

commented Dec 27, 2017

@jlzhou maybe this should be to OCA/server-backend?

@jlzhou

This comment has been minimized.

Copy link

commented Dec 28, 2017

@pedrobaeza Not sure, you are the boss. :)

@pedrobaeza

This comment has been minimized.

Copy link
Member Author

commented Dec 28, 2017

hehe, well, I have the same vote as any contributor. Looking at the split spreadsheet: https://docs.google.com/spreadsheets/d/1Xg95cW4TFMf_Lo5i_CZC_qOOfN8RgxPRc0LJTLTkdUI/edit#gid=0, people had the same doubts about this. Let's see if someone more have a special opinion about this. @lasley @dreispt ? If not, as you are the one making the PR, you can decide by yourself.

@elicoidal

This comment has been minimized.

Copy link

commented Dec 28, 2017

maybe this should be to OCA/server-backend?

I like server-backend but not a big deal as this is just at the limit

@lasley

This comment has been minimized.

Copy link
Member

commented Dec 28, 2017

I agree with server-backend

@astirpe

This comment has been minimized.

Copy link
Member

commented Jan 14, 2018

onchange_helper #1128

@simahawk

This comment has been minimized.

Copy link
Contributor

commented Jan 22, 2018

auth_signup_verify_email OCA/server-auth#8

@astirpe

This comment has been minimized.

Copy link
Member

commented Jan 25, 2018

res_config_settings_enterprise_remove: OCA/server-brand#2

@max3903

This comment has been minimized.

Copy link
Member

commented Feb 9, 2018

Any issue if base_cron_exclusion stays in server-tools? Or do you have a better candidate?

@astirpe

This comment has been minimized.

Copy link
Member

commented Feb 15, 2018

auto_backup #1158

@bodedra

This comment has been minimized.

Copy link
Member

commented Feb 15, 2018

base_cron_exclusion #1156

@youring

This comment has been minimized.

Copy link

commented Apr 12, 2018

base_name_search_improved seems unlisted since v10, and some other very useful module like base_import_odoo, field_char_transformed, qweb_usertime...born for v8, although created after this list, IMHO, should be taken care here or add a note for obsoletion. @pedrobaeza @dreispt @hbrunn Thanks for your contrib and killer-grade tools.

@pedrobaeza

This comment has been minimized.

Copy link
Member Author

commented Apr 12, 2018

The list is copied from v10. If they aren't in that branch when the new branch is created, then they won't be in the list. If you want to migrate one of them directly from v8 to v11, no problem in adding them. For now, we are not going to add them if there are no intentions of migrating them.

@haidd

This comment has been minimized.

Copy link
Contributor

commented May 1, 2018

fetchmail_notify_error_to_sender #1238

@mreficent

This comment has been minimized.

Copy link
Contributor

commented May 14, 2018

auth_totp_password_security OCA/server-auth#18

@aisopuro

This comment has been minimized.

Copy link
Contributor

commented Jun 5, 2018

dead_mans_switch_client #1277

@BT-fgarbely

This comment has been minimized.

Copy link
Contributor

commented Jun 7, 2018

Hello @pedrobaeza
I want to migrate base_suspend_security to v11.0.
I think I have to create the PR in OCA/server-backend

@pedrobaeza

This comment has been minimized.

Copy link
Member Author

commented Jun 7, 2018

@BT-fgarbely yeah, that's the proposed target repository. Go ahead and good luck!

@AMhadhbi

This comment has been minimized.

Copy link
Member

commented Jun 21, 2018

I will work on scheduler_error_mailer

@pedrobaeza

This comment has been minimized.

Copy link
Member Author

commented Jun 21, 2018

Great, noted

@zaoral

This comment has been minimized.

Copy link

commented Jul 5, 2018

I will work on base_name_search_improved

@zaoral

This comment has been minimized.

Copy link

commented Jul 12, 2018

i will work on auth_session_timeout (in to server-auth)

@jlzhou

This comment has been minimized.

Copy link

commented Jul 30, 2018

I am working on menu_technical_info, I think it should be in server-ux repo, but I don't know how to move the module code properly with history, pls advise, thanks.

@gurneyalex

This comment has been minimized.

Copy link
Member

commented Oct 18, 2018

@pedrobaeza is it ok for me to add this module https://github.com/OCA/server-tools/tree/10.0/sql_export to the migration list? Sounds a good candidate for staying in server-tools.

@pedrobaeza

This comment has been minimized.

Copy link
Member Author

commented Oct 18, 2018

Yeah, it's similar to other options:

  • On https://github.com/OCA/server-backend we have several modules for importing from other DBs. Is the same export than import?

  • On OCA/server-ux, there are things that change the interface or add options to the interface (look mass_editing)

so if you decide to continue here, it's OK.

@alexey-pelykh

This comment has been minimized.

Copy link
Contributor

commented Oct 19, 2018

@pedrobaeza what to do if module is in 10.0 and it's needed in 12.0 - port only to 12.0?

@pedrobaeza

This comment has been minimized.

Copy link
Member Author

commented Oct 19, 2018

@alexey-pelykh yes, you don't need to migrate to all intermediate versions (unless you want). On the migration steps, you have to change the source branch from 11.0 to 10.0 for pointing to where the module is migrated.

@misern2

This comment has been minimized.

Copy link

commented Oct 19, 2018

Hi @pedrobaeza .

I want to migrate the base_import_search module to V11 and I don't find it in the google doc.
Which is the best place, server-backend repo ?

@pedrobaeza

This comment has been minimized.

Copy link
Member Author

commented Oct 23, 2018

@misern2 do you mean base_import_match?

@misern2

This comment has been minimized.

Copy link

commented Oct 23, 2018

@pedrobaeza yes!. Sorry for the mistake .. base_import_match ....

@pedrobaeza

This comment has been minimized.

Copy link
Member Author

commented Oct 24, 2018

Yes, OCA/server-backend or this one are good hosters, as can't be easily classified.

@emagdalenaC2i

This comment has been minimized.

Copy link

commented Dec 29, 2018

@pedrobaeza Please, add to the list base_fontawesome - By @etobella - #1361 that is already migrated and merged

Please, add to the list base_name_search_improved - By @zaoral - #1325 that is already migrated and merged

Please, add to the list mail_log_message_to_process that is renamed as -> fetchmail_incoming_log - By @hveficent - #1402 that is already migrated and merged

Please, could you move up key module in the list so that it is sorted correctly in alphabetical order

Please, check onchange_helper - By @astirpe - #1128 because it is already merged

Please, check password_security - By @smurf-U - OCA/server-auth#3

Please, check sentry - By @njeudy - #1082 because it is already merged

Please, add to the list sql_request_abstract - By @hveficent - #1323 that is already migrated and merged

Please, check test_configuration_helper - By @grindtildeath - #1261

@emagdalenaC2i

This comment has been minimized.

Copy link

commented Jun 1, 2019

Please, check auth_from_http_remote_user - By @TDu - OCA/server-auth#7 that is already merged

super_calendar - By @ddufresne - #1173 is closed

@max3903 max3903 pinned this issue Jun 18, 2019

@mvaled

This comment has been minimized.

Copy link

commented Jun 20, 2019

@pedrobaeza

I don't see any PR for module_prototyper. In the document is still listed in the repo.

Is someone working on it, or could I open a PR for it.

Also I don't see even a checkbox for it in #1368. Was this addon split to somewhere else?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.