PHPoAuthLib icon indicating copy to clipboard operation
PHPoAuthLib copied to clipboard

Published 20 hours ago • verified publisher icon daviddesberg

Wrike service added

Open adambalint-srg opened this issue 9 years ago • 1 comments

OAuth service for Wrike Project Management system is added. http://wrike.com

adambalint-srg avatar Mar 03 '16 22:03 adambalint-srg

Reviewed 6 of 6 files at r1. Review status: all files reviewed at latest revision, 5 unresolved discussions.

composer.lock, line 1 [r1] (raw file): Are these changes just part of a composer update and there were no changes to the composer.json?

src/OAuth/OAuth2/Service/Wrike.php, line 29 [r1] (raw file): The /** causes this doc block to only apply to SCOPE_DEFAULT, you should just use /*.

src/OAuth/OAuth2/Service/Wrike.php, line 53 [r1] (raw file): Please move "3" into a constant.

src/OAuth/OAuth2/Service/Wrike.php, line 139 [r1] (raw file): This file mixes tabs and spaces, please reformat it to just use 4 spaces for indentation.

tests/Unit/OAuth2/Service/WrikeTest.php, line 145 [r1] (raw file): Also mixing tabs and spaces here, please replace all the tabs with 4 spaces.

Comments from Reviewable

elliotchance avatar May 05 '16 08:05 elliotchance