-
Notifications
You must be signed in to change notification settings - Fork 28
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
loaders.load_csv
: allow relative paths w/ ODOO_DATA_PATH
#36
Conversation
7f618df
to
ba1f83d
Compare
:param model: Odoo model name or model klass from env | ||
:param path: absolute or relative path to CSV file. | ||
If a relative path is given you must provide a value for | ||
`ODOO_DATA_PATH` in your environment |
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.
"or set the --odoo-data-path
option"
anthem/cli.py
Outdated
@@ -53,9 +57,21 @@ def main(): | |||
'configuration file and -d for the database name. Those must be at ' | |||
'the end of the arguments' | |||
) | |||
parser.add_argument( | |||
'odoo-data-path', | |||
nargs=argparse.REMAINDER, |
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.
REMAINDER
is to take all the "unknown" remaining args, which will be passed down to odoo, only odoo-args
should use it. This argument should probably only have action='store_true'
action='store'
(the default) and no nargs
ba1f83d
to
f2e13a3
Compare
Travis says:
|
f2e13a3
to
b374115
Compare
@guewen damn, somehow I lost the |
b374115
to
f2cbbd6
Compare
No description provided.