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
[ADD] document_url: A module that allows to put URLs as attached documents. #17
Conversation
############################################################################## | ||
# | ||
# OpenERP, Open Source Management Solution | ||
# Copyright (C) 2004-2010 Tiny SPRL (<http://tiny.be>). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this correct and up to date?
@gurneyalex, I have addressed your remarks, except one, that I have commented. |
@@ -13,10 +14,8 @@ install: | |||
- git clone https://github.com/OCA/maintainer-quality-tools.git ${HOME}/maintainer-quality-tools | |||
- export PATH=${HOME}/maintainer-quality-tools/travis:${PATH} | |||
- travis_install_nightly ${VERSION} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
giving the version as commandline argument is deprecated from my understanding
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right. I'll update it.
2a04fd6
to
52d7eb8
Compare
redraw: function() { | ||
var self = this; | ||
this._super.apply(this, arguments); | ||
self.$el.find('.oe_sidebar_add_attachment').after(QWeb.render('AddUrlDocumentItem', {widget: self})) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why don't you inject the element by extending Sidebar in your qweb file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My knowledge in JS is very limited and I copied/pasted the code from Google documents standard module.
@hbrunn , can you please re-review this? |
👍 |
@pedrobaeza sorry for the delay, I'm fine with the code as it is 👍 |
[ADD] document_url: A module that allows to put URLs as attached documents.
No description provided.