-
Notifications
You must be signed in to change notification settings - Fork 23
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
Update widgets, refactor and code clean up #133
Conversation
f8a2be0
to
72270a8
Compare
Clean up and refactor. More information: * https://developer.linkedin.com/plugins/share * https://developer.linkedin.com/docs/share-on-linkedin
We need support for default value in get_registry_record(). Also, remove unnecessary [blobs] extra on plone.namedfile.
72270a8
to
5cddc29
Compare
|
||
# TODO: check if Pinterest can use Open Graph instead |
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.
Actually Pinterest uses Open Graph; This method is used just when DnT is activated, the image url needs to be sent as parameter to Pinterest.
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.
ok, I'm going to make some more changes according to that.
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.
Great Job!
Clean up and refactor. More information: * https://developers.pinterest.com/docs/widgets/save/ * https://developers.pinterest.com/docs/rich-pins/reference/
Clean up and refactor. More information: * https://developers.google.com/+/web/share/
5cddc29
to
0cf9080
Compare
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 didn't even give time to us to review, rs. But looks good, specially the plone.api upgrade to avoid InvalidParameterError
checks all over the place.
typebutton
used to persist 'horizontal' but now you only check for 'vertical', you don't need an upgradStep here, right?
we need to finish it this week :-) no, in fact I want to get rid of that at some point as is a collections of hacks: all plugins treat it on a different way. |
This should make plugins faster and consume less memory.
refs. #117