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

9.0 cmis field #3

Closed
wants to merge 125 commits into from
Closed

9.0 cmis field #3

wants to merge 125 commits into from

Conversation

lmignon
Copy link
Member

@lmignon lmignon commented Sep 2, 2016

No description provided.

Sandy Carter and others added 30 commits September 2, 2014 15:14
…irectory_of_write, username and password fields to be required
In order to get visibility on https://www.odoo.com/apps the OCA board has
decided to add the OCA as author of all the addons maintained as part of the
association.
'cmis_name_get': 'name_get',
'allow_create': True,
'allow_delete': False,
'cmis_path': None,
Copy link
Member

Choose a reason for hiding this comment

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

create_root_get ?

Copy link
Member

Choose a reason for hiding this comment

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

This could be a method too, since the most common override is changing the root.

Copy link
Member Author

Choose a reason for hiding this comment

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

@sbidoul
3 parameters can be passed to the cmis methot createFolder

  • the parent object id, (required)
  • the content name (required)
  • and a dict for properties (optional)

What do you think of the idea of having one parameter for each item

  • create_root_get
  • create_name_get
  • create_properties_get

Copy link
Member

Choose a reason for hiding this comment

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

Yes, good idea.

Get the backend desciption to put in the backend description from the cmis.backend model
@coveralls
Copy link

coveralls commented Sep 5, 2016

Coverage Status

Changes Unknown when pulling f8912b8 on 9.0-cmis-field into * on 9.0-empty*.

@coveralls
Copy link

coveralls commented Sep 5, 2016

Coverage Status

Changes Unknown when pulling aea49bd on 9.0-cmis-field into * on 9.0-empty*.

@codecov-io
Copy link

codecov-io commented Sep 5, 2016

Current coverage is 55.52% (diff: 55.52%)

No coverage report found for 9.0-empty at ca8b565.

Powered by Codecov. Last update ca8b565...aea49bd

@coveralls
Copy link

coveralls commented Sep 7, 2016

Coverage Status

Changes Unknown when pulling e6f127f on 9.0-cmis-field into * on 9.0-empty*.

@lmignon
Copy link
Member Author

lmignon commented Sep 7, 2016

superseded by #3

@lmignon lmignon closed this Sep 7, 2016
@lmignon lmignon deleted the 9.0-cmis-field branch September 9, 2016 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
8 participants