<table><tr><td style="">barchiesi updated this revision to Diff 57191.<br />barchiesi added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D20886">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D20886#458007" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D20886#458007</a>, <a href="https://phabricator.kde.org/p/dvratil/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@dvratil</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Since this changes the API, we must also bump the version in CMakeLists.txt. KIO GDrive will then have to have an <tt style="background: #ebebeb; font-size: 13px;">#ifdef</tt> on  LibKGAPI version to use either the old or the new approach, based on against what version of LibKGAPI it is compiled. This LibKGAPI API will not be publicly available until August (KDE Applications 19.08 release).</p>

<p>Another option is to have KIO GDrive master depend on the master of LibKGAPI and not doing any KIO GDrive release from master until LibKGAPI 19.08 is released (but I doubt <a href="https://phabricator.kde.org/p/elvisangelaccio/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@elvisangelaccio</a> would be too happy about that :-) )</p></div>
</blockquote>

<p>I guess we should let <a href="https://phabricator.kde.org/p/elvisangelaccio/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@elvisangelaccio</a> decide, I believe the first approach is better because it allows building a newer KIO GDrive against older versions of LibKGAPI.</p>

<p>I think this revision needs some more work anyways, I would like to find an elegant way of applying the fields query param in the case of lists. Both <tt style="background: #ebebeb; font-size: 13px;">FileFetchJob</tt> and <tt style="background: #ebebeb; font-size: 13px;">TeamdriveFetchJob</tt> (as seen in my last update) need to manually handle applying filters to <tt style="background: #ebebeb; font-size: 13px;">items</tt>.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R477 KGAPI Library</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D20886?vs=57174&id=57191">https://phabricator.kde.org/D20886?vs=57174&id=57191</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20886">https://phabricator.kde.org/D20886</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/drive/aboutfetchjobtest.cpp<br />
autotests/drive/data/about_fetch_limited_fields_request.txt<br />
autotests/drive/data/about_fetch_limited_fields_response.txt<br />
autotests/drive/data/about_limited_fields.json<br />
examples/teamdrive/mainwindow.cpp<br />
src/core/job.cpp<br />
src/core/job.h<br />
src/core/job_p.h<br />
src/drive/about.cpp<br />
src/drive/about.h<br />
src/drive/file.cpp<br />
src/drive/file.h<br />
src/drive/filefetchjob.cpp<br />
src/drive/filefetchjob.h<br />
src/drive/teamdrive.cpp<br />
src/drive/teamdrive.h<br />
src/drive/teamdrivefetchjob.cpp<br />
src/drive/teamdrivefetchjob.h</div></div></div><br /><div><strong>To: </strong>barchiesi, dvratil<br /><strong>Cc: </strong>elvisangelaccio, kde-pim, LibKGAPI, barchiesi, gennad, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>