Merge lp://qastaging/~axiometry/shutter/devel into lp://qastaging/shutter

Proposed by Marco van Hilst
Status: Merged
Merge reported by: Mario Kemper (Romario)
Merged at revision: not available
Proposed branch: lp://qastaging/~axiometry/shutter/devel
Merge into: lp://qastaging/shutter
Diff against target: 989 lines (+905/-17)
2 files modified
share/shutter/resources/modules/WebService/Dropbox.pm (+881/-0)
share/shutter/resources/system/upload_plugins/upload/Dropbox.pm (+24/-17)
To merge this branch: bzr merge lp://qastaging/~axiometry/shutter/devel
Reviewer Review Type Date Requested Status
Mario Kemper (Romario) Approve
Review via email: mp+228554@code.qastaging.launchpad.net

Description of the change

Update Dropbox plugin to generate direct links via the /media Dropbox API.
 * Switch to WebService::Dropbox
 * Use WebService::Dropbox->files_put to upload files
 * Use WebService::Dropbox->media to generate a direct link to the screenshot file
 * Uses LWP backend just as Net::Dropbox::API did

Add support for changing the upload folder.
 * Uses the config in ~/.dropbox-api-config for backwards compatibility, and adds the key 'upload_folder'

To post a comment you must log in.
Revision history for this message
Mario Kemper (Romario) (mario-kemper) wrote :

Very good! Thanks for your work.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: