Skip to content

[11.0][MIG] Knowledge Migration#149

Closed
fayezqandeel wants to merge 28 commits into
OCA:11.0from
fayezqandeel:11.0-mig-knowledge
Closed

[11.0][MIG] Knowledge Migration#149
fayezqandeel wants to merge 28 commits into
OCA:11.0from
fayezqandeel:11.0-mig-knowledge

Conversation

@fayezqandeel

Copy link
Copy Markdown
Contributor

No description provided.

naousse and others added 26 commits December 19, 2017 14:34
reamed.rst added
version number corrected
res_config.py is now  respecting the new odoo API
Filter on "real"/user documents (no odoo assets), filter on res_model,
filter on indexed content.
* [MIG] Migration to version 10.0
* [FIX] Comments and CI errors
* [FIX] ValueError: External ID not found in the system: base.menu_base_partner
* [FIX] ValueError: Wrong value for ir.actions.act_window.target: 'inlineview'
* [FIX] Based on @lasley comments
* [FIX] External ID not found in the system: base.group_document_user
@fayezqandeel

Copy link
Copy Markdown
Contributor Author

@pedrobaeza @mreficent I think now its fine, could check please?

@pedrobaeza pedrobaeza mentioned this pull request Dec 19, 2017
12 tasks
@jalzaga jalzaga mentioned this pull request Apr 2, 2018
1 task
Comment thread knowledge/README.rst

.. image:: https://odoo-community.org/website/image/ir.attachment/5784_f2813bd/datas
:alt: Try me on Runbot
:target: https://runbot.odoo-community.org/runbot/repo/118/10.0

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/118/11.0

Comment thread knowledge/README.rst
If you spotted it first, help us smashing it by providing a detailed and welcomed feedback `here <https://github.com/OCA/
knowledge/issues/new?body=module:%20
knowledge%0Aversion:%20
10.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

11.0

Comment thread knowledge/README.rst
* Savoir-faire Linux <support@savoirfairelinux.com>
* Gervais Naoussi <gervaisnaoussi@gmail.com>
* Leonardo Donelli <leonardo.donelli@monksoftware.it>
* Maxime Chambreuil <mchambreuil@ursainfosystems.com>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@fayezqandeel Please add yourself to this list

@@ -0,0 +1,12 @@
<?xml version="1.0" encoding="utf-8"?>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

No need for the encoding any more.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes for the XML files. It's not needed on Python files in Python 3.

@pedrobaeza

Copy link
Copy Markdown
Member

Is this including a configuration flag for installing missing modules like CMIS? I don't see that like a good idea.

@max3903

max3903 commented Apr 10, 2018

Copy link
Copy Markdown
Member

@pedrobaeza Yes it does. I think it's a good way to promote other related modules.

@pedrobaeza

Copy link
Copy Markdown
Member

Yes, but not know that the rest of the modules are not migrated (and maybe they will be never migrated) and thus they throw an error. This should be detected dynamically if there's a module in the module list with the proper name for showing it.

@max3903

max3903 commented Apr 16, 2018

Copy link
Copy Markdown
Member

@fayezqandeel Are you going to fix this pull request? Please let us know as there is a lot of other modules depending on this one. Thanks.

@max3903 max3903 added this to the 11.0 milestone Apr 16, 2018
@cubells

cubells commented May 8, 2018

Copy link
Copy Markdown
Member

Hi @fayezqandeel

I want to finish this pr. Can I push to this branch?

@SalahAdDin

Copy link
Copy Markdown
Contributor

@cubells we need too.

@cubells

cubells commented Jun 4, 2018

Copy link
Copy Markdown
Member

@SalahAdDin I'll create a new pr.

@SalahAdDin

Copy link
Copy Markdown
Contributor

@cubells ok, i'm waiting.

@cubells cubells mentioned this pull request Jun 19, 2018
@cubells

cubells commented Jun 19, 2018

Copy link
Copy Markdown
Member

@saadthaifa new pr is here: #170 and supersede this.

@pedrobaeza pedrobaeza closed this Jun 19, 2018
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.

9 participants