[kdenlive] Review Request 126796: online resource -logs into users freesound accounts to get permission for HQ audio. Fixes archive.org video search new dependancy KF5WebKit. (libkf5webkit5-dev)

Roger Morton ttguy1 at gmail.com
Thu Feb 4 12:39:36 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126796/
-----------------------------------------------------------

(Updated Feb. 4, 2016, 12:39 p.m.)


Review request for Kdenlive and Vincent Pinon.


Changes
-------

moved saving of tokens to settings file. added missing licence info to new files


Repository: kdenlive


Description
-------

online resource -logs into users freesound accounts to get permission for HQ audio. Fixes archive.org video search new dependancy KF5WebKit. (libkf5webkit5-dev)


Diffs
-----

  src/CMakeLists.txt 87d29d2 
  src/qt-oauth-lib/logindialog.h PRE-CREATION 
  src/qt-oauth-lib/logindialog.cpp PRE-CREATION 
  src/qt-oauth-lib/oauth2.h PRE-CREATION 
  src/qt-oauth-lib/oauth2.cpp PRE-CREATION 
  src/ui/freesound_ui.ui 55a453d 
  src/utils/abstractservice.h 0a6a4a7 
  src/utils/archiveorg.h 5cdc84f 
  src/utils/archiveorg.cpp 1be4d6f 
  src/utils/freesound.h f72d145 
  src/utils/freesound.cpp 932c47f 
  src/utils/openclipart.h 7a4df1e 
  src/utils/openclipart.cpp adbba68 
  src/utils/resourcewidget.h c171e3c 
  src/utils/resourcewidget.cpp 6453f7f 

Diff: https://git.reviewboard.kde.org/r/126796/diff/


Testing
-------

Built. Tested file downloads from freesound. Auhenticates to freesound. keeps authenticated until you delete $HOME/.config/kde.org/kdenlive.conf 
Archive.org video search is now working. Display animiated .gif as the preview file from Archive.org videos.
No longer have to click search button twice to get it to do the search the first time it runs
Option to not use personal freesound account and just use HQ preview file instead.
Downloads low quality preview file in background and does not notify you of it finishing.


File Attachments (updated)
----------------

tokens to settings patch
  https://git.reviewboard.kde.org/media/uploaded/files/2016/02/04/9529c467-2fa8-459e-84c5-a899ca4275ee__0002-move-token-storage-in-freesound-to-settings-file.patch


Thanks,

Roger Morton

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdenlive/attachments/20160204/fa9d6de1/attachment.html>


More information about the kdenlive mailing list