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

Properties cleanup #686

Merged
merged 34 commits into from
Aug 30, 2016
Merged

Properties cleanup #686

merged 34 commits into from
Aug 30, 2016

Conversation

fvennetier
Copy link
Member

@fvennetier fvennetier commented Aug 30, 2016

No description provided.

jfsmig and others added 30 commits August 5, 2016 09:54
Also removes unused macros and useless comments
COMPATIBILITY BREAK

* proxy: format modified for properties-related routes
* asn1c: dropped a lot of useless messages dedicated to properties
         variants. From 5 methods, just one remains (JSON).
* unused code removed
PROTOCOLAR COMPATIBILITY BREAK
... when creating entities, setting properties, getting them.
... whatever the entity (user, container, content) the format is the
same at the proxy level and at the backend service level.
core: The properties indicate in upload are set in the upload_commit step
core: modify set_properties and get_properties to respect new format
@fvennetier fvennetier merged commit 5fa5ada into master Aug 30, 2016
@fvennetier fvennetier deleted the properties-cleanup branch August 30, 2016 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants