{"action":"created","comment":{"url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/pulls/comments/63219958","id":63219958,"diff_hunk":"@@ -0,0 +1,34 @@\n+﻿using System;\n+using System.Collections.Generic;\n+using System.Linq;\n+using System.Net.Http;\n+using System.Threading.Tasks;\n+using GitHubExtention.Preferences.WebApi.Constants;\n+using GitHubExtention.Preferences.WebApi.Extensions;\n+using GitHubExtention.Preferences.WebApi.Models;\n+using GitHubExtention.Preferences.WebApi.Queries;\n+using Microsoft.WindowsAzure.Storage.Blob;\n+\n+namespace GitHubExtention.Preferences.WebApi\n+{\n+    public static class AvatarOperations\n+    {\n+        public static async Task<string> GetNewAvatarUrl(this IAzureContainerQuery container, string url)","path":"DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/AvatarOperations.cs","position":16,"original_position":16,"commit_id":"0c48539fac6fe2fe3aff2df30f155f5bfaea9cca","original_commit_id":"0c48539fac6fe2fe3aff2df30f155f5bfaea9cca","user":{"login":"IrynaRobulets","id":17322068,"avatar_url":"https://avatars.githubusercontent.com/u/17322068?v=3","gravatar_id":"","url":"https://api.github.com/users/IrynaRobulets","html_url":"https://github.com/IrynaRobulets","followers_url":"https://api.github.com/users/IrynaRobulets/followers","following_url":"https://api.github.com/users/IrynaRobulets/following{/other_user}","gists_url":"https://api.github.com/users/IrynaRobulets/gists{/gist_id}","starred_url":"https://api.github.com/users/IrynaRobulets/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/IrynaRobulets/subscriptions","organizations_url":"https://api.github.com/users/IrynaRobulets/orgs","repos_url":"https://api.github.com/users/IrynaRobulets/repos","events_url":"https://api.github.com/users/IrynaRobulets/events{/privacy}","received_events_url":"https://api.github.com/users/IrynaRobulets/received_events","type":"User","site_admin":false},"body":"This method calls when user firstly register in our system and only way to get image url is to send request to GitHub get image,  save this  image to blobStorage and finally get the image url. Maybe it better to make two method one for getting image file and second for saving this image to blobStorage.","created_at":"2016-05-13T17:25:41Z","updated_at":"2016-05-13T17:25:41Z","html_url":"https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r63219958","pull_request_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/pulls/138","_links":{"self":{"href":"https://api.github.com/repos/ITsvetkoFF/Kv-013/pulls/comments/63219958"},"html":{"href":"https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r63219958"},"pull_request":{"href":"https://api.github.com/repos/ITsvetkoFF/Kv-013/pulls/138"}}},"pull_request":{"url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/pulls/138","id":69180281,"html_url":"https://github.com/ITsvetkoFF/Kv-013/pull/138","diff_url":"https://github.com/ITsvetkoFF/Kv-013/pull/138.diff","patch_url":"https://github.com/ITsvetkoFF/Kv-013/pull/138.patch","issue_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/issues/138","number":138,"state":"open","locked":false,"title":"Feature/upload photo backend","user":{"login":"IrynaRobulets","id":17322068,"avatar_url":"https://avatars.githubusercontent.com/u/17322068?v=3","gravatar_id":"","url":"https://api.github.com/users/IrynaRobulets","html_url":"https://github.com/IrynaRobulets","followers_url":"https://api.github.com/users/IrynaRobulets/followers","following_url":"https://api.github.com/users/IrynaRobulets/following{/other_user}","gists_url":"https://api.github.com/users/IrynaRobulets/gists{/gist_id}","starred_url":"https://api.github.com/users/IrynaRobulets/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/IrynaRobulets/subscriptions","organizations_url":"https://api.github.com/users/IrynaRobulets/orgs","repos_url":"https://api.github.com/users/IrynaRobulets/repos","events_url":"https://api.github.com/users/IrynaRobulets/events{/privacy}","received_events_url":"https://api.github.com/users/IrynaRobulets/received_events","type":"User","site_admin":false},"body":"<a href='#crh-start'></a><a href='#crh-data-%7B%22processed%22%3A%20%5B%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62653996%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62654257%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62654314%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62654516%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62654602%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62654694%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62654898%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62654982%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62655310%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62690678%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62691295%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62703064%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62703150%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62706624%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62707619%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62707696%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62711823%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62712554%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62712634%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63206150%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63206731%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63207315%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63207901%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63208190%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63208445%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63208716%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63208874%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63209219%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63209255%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63209487%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63211198%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63211488%22%2C%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63215186%22%5D%2C%20%22comments%22%3A%20%7B%22Pull%20014860779d7aa5040ca650c94cf285d073d7caa6%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/AvatarOperations.cs%2013%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62654314%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22Strange.%20Why%20you%20prevent%20using%20async/await%20functionality%3F%22%2C%20%22created_at%22%3A%20%222016-05-10T11%3A24%3A21Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%2C%20%7B%22body%22%3A%20%22I%20have%20changed%20it%20to%20async%22%2C%20%22created_at%22%3A%20%222016-05-10T16%3A16%3A37Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/17322068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/IrynaRobulets%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/AvatarOperations.cs%3AL1-32%22%7D%2C%20%22Pull%20014860779d7aa5040ca650c94cf285d073d7caa6%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Extensions/HttpResponseExtensions.cs%2012%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62654982%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22Can%20you%20explain%20why%20this%20method%20is%20not%20extension%20method%20with%20a%20single%20responsibility%3F%22%2C%20%22created_at%22%3A%20%222016-05-10T11%3A30%3A39Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%2C%20%7B%22body%22%3A%20%22Now%20it%27s%20with%20single%20responsibility.%22%2C%20%22created_at%22%3A%20%222016-05-10T16%3A37%3A42Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/17322068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/IrynaRobulets%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Extensions/HttpResponseExtensions.cs%3AL1-19%22%7D%2C%20%22Pull%20014860779d7aa5040ca650c94cf285d073d7caa6%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs%2045%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62654898%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22Please%20don%27t%20mix%20persistence%20layer%20with%20Azure%20complexity...It%20should%20be%20separarted%20btw%20DbContext%20also%20support%20async%20operations.%22%2C%20%22created_at%22%3A%20%222016-05-10T11%3A29%3A36Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs%3AL1-77%22%7D%2C%20%22Pull%200c48539fac6fe2fe3aff2df30f155f5bfaea9cca%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Formatters/FileMediaFormatter.cs%2057%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63208190%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22Move%20to%20extesion%20methods.%22%2C%20%22created_at%22%3A%20%222016-05-13T16%3A01%3A40Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Formatters/FileMediaFormatter.cs%3AL1-86%22%7D%2C%20%22Pull%20c2f6558c0b2f7c4aa483ff4f76aeb0a17dbb3403%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Azure/AzureBlobStorageMultipartProvider%2520.cs%2044%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62712554%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22You%20can%20return%20file%20location%20from%20method%20UpdateBlob.%20It%27s%20covered%20by%20case%20when%20after%20uploading%20file%20you%20will%20get%20path%20to%20it%20location.%22%2C%20%22created_at%22%3A%20%222016-05-10T17%3A13%3A02Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Azure/AzureBlobStorageMultipartProvider%20.cs%3AL1-61%22%7D%2C%20%22Pull%200c48539fac6fe2fe3aff2df30f155f5bfaea9cca%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Extensions/HttpResponseExtensions.cs%2015%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63209255%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22I%20like%20such%20small%20and%20concrete%20extension%20methods.%22%2C%20%22created_at%22%3A%20%222016-05-13T16%3A08%3A51Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Extensions/HttpResponseExtensions.cs%3AL1-20%22%7D%2C%20%22Pull%200c48539fac6fe2fe3aff2df30f155f5bfaea9cca%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Queries/PreferencesQuery.cs%2031%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63207315%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22May%20be%20it%27s%20better%20to%20pass%20User%20entity%20with%20updated%20state%3F%20Because%20now%20I%20don%27t%20see%20any%20reson%20to%20pass%20this%20additional%20parameter%20as%20URL.%22%2C%20%22created_at%22%3A%20%222016-05-13T15%3A56%3A53Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Queries/PreferencesQuery.cs%3AL1-36%22%7D%2C%20%22Pull%20014860779d7aa5040ca650c94cf285d073d7caa6%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Package/PreferencesPackage.cs%2034%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62655310%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22You%20really%20need%20this%20registration%3F%22%2C%20%22created_at%22%3A%20%222016-05-10T11%3A33%3A34Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%2C%20%7B%22body%22%3A%20%22I%27m%20not%20sure%20exectly%2C%20I%20guess%20if%20user%20not%20null%20he%20has%20permissions%20to%20update%20photo.%20I%20check%20permissions%20because%20they%20can%20be%20changed%20in%20the%20azure%20out%20of%20code.%20So%2C%20if%20supose%20that%20permissions%20won%27t%20be%20changed%20in%20Azure%20this%20checking%20have%20no%20sence.%20Do%20I%20need%20get%20it%20away%3F%22%2C%20%22created_at%22%3A%20%222016-05-10T16%3A43%3A33Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/17322068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/IrynaRobulets%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Package/PreferencesPackage.cs%3AL1-43%22%7D%2C%20%22Pull%20014860779d7aa5040ca650c94cf285d073d7caa6%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs%2023%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62654694%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22You%20really%20need%20expose%20access%20to%20all%20persictence%20layer%3F%22%2C%20%22created_at%22%3A%20%222016-05-10T11%3A28%3A03Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%2C%20%7B%22body%22%3A%20%22Nope%2C%20I%27ve%20changed%20it%22%2C%20%22created_at%22%3A%20%222016-05-10T16%3A17%3A09Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/17322068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/IrynaRobulets%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs%3AL1-77%22%7D%2C%20%22Pull%200c48539fac6fe2fe3aff2df30f155f5bfaea9cca%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/AvatarOperations.cs%2016%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63206150%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22Method%20with%20a%20Get%20prefix%20but%20inside%20the%20body%20it%3Bs%20call%20UpdateBlob%3F%20Can%20you%20clarify%20this....%22%2C%20%22created_at%22%3A%20%222016-05-13T15%3A50%3A03Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/AvatarOperations.cs%3AL1-35%22%7D%2C%20%22Pull%20014860779d7aa5040ca650c94cf285d073d7caa6%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Azure/AzureBlobStorageMultipartProvider%2520.cs%2010%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62654516%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22Please%20hide%20all%20details%20of%20implementation%20by%20interface.%20SOLID...Class%20should%20be%20acceptable%20for%20extension%20not%20for%20MODIFICATION%21%20The%20second%20part%20you%20should%20not%20rely%20on%20implementation%20of%20MultipartFileStreamProvider%20it%20can%20be%20changed...%22%2C%20%22created_at%22%3A%20%222016-05-10T11%3A26%3A19Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%2C%20%7B%22body%22%3A%20%22If%20I%20use%20interface%20instead%20of%20AzureBlobStorageMultipartProvider.%20I%20can%27t%20use%20ReadAsMultipartAsync%3CAzureBlobStorageMultipartProvider%3E%28_multipartStreamProvider%29%20becuase%20this%20method%20requires%20paramater%20refernces%20to%20abstract%20class%20MultipartStreamProvider.%20As%20a%20result%20I%20need%20to%20make%20my%20own%20conten%20reading%20provider.%20Am%20I%20understand%20you%3F%22%2C%20%22created_at%22%3A%20%222016-05-10T15%3A12%3A55Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/17322068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/IrynaRobulets%22%7D%7D%2C%20%7B%22body%22%3A%20%22Interesting%20case.%20But%20why%20you%20did%27t%20accept%20parameters%20into%20action%20body%3F%20Instear%20using%20Request%20property%20of%20ApiController%3F%20As%20I%20understand%20Web%20Api%20allows%20to%20have%20mixed%20parameters%20inside%20action%20signature.%22%2C%20%22created_at%22%3A%20%222016-05-10T17%3A08%3A17Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Azure/AzureBlobStorageMultipartProvider%20.cs%3AL1-61%22%7D%2C%20%22Pull%200c48539fac6fe2fe3aff2df30f155f5bfaea9cca%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs%2074%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63209219%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22Btw%20it%20ca%20be%20moved%20to%20UserExtension%20method%20to%20chechking%20url%20validity.%22%2C%20%22created_at%22%3A%20%222016-05-13T16%3A08%3A33Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs%3AL1-81%22%7D%2C%20%22Pull%20014860779d7aa5040ca650c94cf285d073d7caa6%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Azure/AzureBlobStorageMultipartProvider%2520.cs%2015%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62654602%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22Nice%20injection...Why%20you%20are%20not%20using%20the%20same%20approach%20with%20MultipartProvider%3F%22%2C%20%22created_at%22%3A%20%222016-05-10T11%3A27%3A05Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%2C%20%7B%22body%22%3A%20%22Do%20you%20mean%20MultipartFileStreamProvider%3F%20Why%20I%20need%20to%20inject%20it%2C%20there%20is%20no%20place%20where%20I%20create%20an%20instance.%20It%27s%20only%20inheritance%20from%20this%20class.%22%2C%20%22created_at%22%3A%20%222016-05-10T15%3A15%3A55Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/17322068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/IrynaRobulets%22%7D%7D%2C%20%7B%22body%22%3A%20%22Yep%2C%20need%20to%20think%20how%20to%20better%20use%20this%20Provider.%22%2C%20%22created_at%22%3A%20%222016-05-10T17%3A13%3A38Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Azure/AzureBlobStorageMultipartProvider%20.cs%3AL1-61%22%7D%2C%20%22Pull%200c48539fac6fe2fe3aff2df30f155f5bfaea9cca%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/AvatarOperations.cs%2025%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63208716%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22May%20be%20you%20should%20first%20check%20is%20fomat%20valid%20and%20the%20setting%20up%20all%20properties%3F%22%2C%20%22created_at%22%3A%20%222016-05-13T16%3A05%3A13Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%2C%20%7B%22body%22%3A%20%22Yeap%2C%20it%27s%20more%20logical%22%2C%20%22created_at%22%3A%20%222016-05-13T16%3A21%3A36Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/17322068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/IrynaRobulets%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/AvatarOperations.cs%3AL1-35%22%7D%2C%20%22Pull%200c48539fac6fe2fe3aff2df30f155f5bfaea9cca%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Formatters/FileMediaFormatter.cs%2070%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63208445%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22Can%20this%20method%20simplified%3F%22%2C%20%22created_at%22%3A%20%222016-05-13T16%3A03%3A22Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Formatters/FileMediaFormatter.cs%3AL1-86%22%7D%2C%20%22Pull%200c48539fac6fe2fe3aff2df30f155f5bfaea9cca%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Models/FileModel.cs%2011%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63207901%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22We%20need%20this%20alone%20constructor%3F%22%2C%20%22created_at%22%3A%20%222016-05-13T16%3A00%3A38Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Models/FileModel.cs%3AL1-29%22%7D%2C%20%22Pull%20014860779d7aa5040ca650c94cf285d073d7caa6%20DevelopGitHub/GitHubExtension/GitHubExtension.Security.WebApi/Controllers/AccountController.cs%2033%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62654257%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22Hmm...Explain%20why%20you%20are%20is%20not%20awaiting%20your%20avatar%20url%3F%22%2C%20%22created_at%22%3A%20%222016-05-10T11%3A23%3A54Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%2C%20%7B%22body%22%3A%20%22I%27m%20waiting%20it%20now.%20%22%2C%20%22created_at%22%3A%20%222016-05-10T16%3A44%3A00Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/17322068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/IrynaRobulets%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtension.Security.WebApi/Controllers/AccountController.cs%3AL63-70%22%7D%2C%20%22Pull%200c48539fac6fe2fe3aff2df30f155f5bfaea9cca%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs%2044%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63208874%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22Please%20rename%20to%20something%20like%20IsUrlValidbecause%20I%20don%27t%20see%20that%20we%20work%20with%20actual%20files.%20Only%20urls.%20Correct%3F%22%2C%20%22created_at%22%3A%20%222016-05-13T16%3A06%3A13Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%2C%20%7B%22body%22%3A%20%221.%20Done%5Cr%5Cn2.%20I%20guess%20that%20method%20name%20is%20not%20the%20best.%20The%20idea%20is%20to%20check%20if%20the%20avatar%20url%20have%20changed%20than%20we%20need%20to%20update%20this%20field%20in%20database%20and%20delete%20unnecessary%20file%20from%20blobstorage.%20If%20url%20the%20same%2C%20there%20is%20no%20point%20to%20update%20url%20in%20database.%20Maybe%20the%20method%20name%20like%20IsUrlSimilar%20will%20be%20more%20clear.%22%2C%20%22created_at%22%3A%20%222016-05-13T16%3A51%3A54Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/17322068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/IrynaRobulets%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs%3AL1-81%22%7D%2C%20%22Pull%200c48539fac6fe2fe3aff2df30f155f5bfaea9cca%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Queries/PreferencesQuery.cs%2019%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63206731%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22I%20see%20a%20small%20smell...You%20are%20exposing%20Users%20property%20and%20also%20declare%20GetUser%20method%20in%20interface%3F%20Do%20not%20many%20methods%20inside%20interface%3F%20Where%20our%20flexibility%20related%20to%20IOrderedQueryable%3F%22%2C%20%22created_at%22%3A%20%222016-05-13T15%3A53%3A24Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Queries/PreferencesQuery.cs%3AL1-36%22%7D%2C%20%22Pull%20014860779d7aa5040ca650c94cf285d073d7caa6%20DevelopGitHub/GitHubExtension/GitHubExtension.Security.WebApi/Controllers/AccountController.cs%2020%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r62653996%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22You%20really%20need%20instance%20type%20not%20interface%3F%22%2C%20%22created_at%22%3A%20%222016-05-10T11%3A21%3A39Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtension.Security.WebApi/Controllers/AccountController.cs%3AL31-49%22%7D%2C%20%22Pull%200c48539fac6fe2fe3aff2df30f155f5bfaea9cca%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs%2035%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/ITsvetkoFF/Kv-013/pull/138%23discussion_r63209487%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22So%20I%20suggest%20all%20magic%20happens%20inside%20formatter%20and%20here%20you%20already%20have%20filled%20all%20necessesary%20fields%3F%22%2C%20%22created_at%22%3A%20%222016-05-13T16%3A10%3A30Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/6068068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/TerraVenil%22%7D%7D%2C%20%7B%22body%22%3A%20%22Yes%22%2C%20%22created_at%22%3A%20%222016-05-13T16%3A23%3A40Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/17322068%3Fv%3D3%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/IrynaRobulets%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs%3AL1-81%22%7D%7D%7D'></a>\n<a href='https://www.codereviewhub.com/'><img src='http://www.codereviewhub.com/site/github-bar.png' height=40></a>\n- [ ] <a href='#crh-comment-Pull 014860779d7aa5040ca650c94cf285d073d7caa6 DevelopGitHub/GitHubExtension/GitHubExtension.Security.WebApi/Controllers/AccountController.cs 20'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r62653996'>File: DevelopGitHub/GitHubExtension/GitHubExtension.Security.WebApi/Controllers/AccountController.cs:L31-49</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> You really need instance type not interface?\n- [ ] <a href='#crh-comment-Pull 014860779d7aa5040ca650c94cf285d073d7caa6 DevelopGitHub/GitHubExtension/GitHubExtension.Security.WebApi/Controllers/AccountController.cs 33'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r62654257'>File: DevelopGitHub/GitHubExtension/GitHubExtension.Security.WebApi/Controllers/AccountController.cs:L63-70</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> Hmm...Explain why you are is not awaiting your avatar url?\n- <a href='https://github.com/IrynaRobulets'><img border=0 src='https://avatars.githubusercontent.com/u/17322068?v=3' height=16 width=16'></a> I'm waiting it now.\n- [ ] <a href='#crh-comment-Pull 014860779d7aa5040ca650c94cf285d073d7caa6 DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/AvatarOperations.cs 13'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r62654314'>File: DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/AvatarOperations.cs:L1-32</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> Strange. Why you prevent using async/await functionality?\n- <a href='https://github.com/IrynaRobulets'><img border=0 src='https://avatars.githubusercontent.com/u/17322068?v=3' height=16 width=16'></a> I have changed it to async\n- [ ] <a href='#crh-comment-Pull 014860779d7aa5040ca650c94cf285d073d7caa6 DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Azure/AzureBlobStorageMultipartProvider%20.cs 10'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r62654516'>File: DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Azure/AzureBlobStorageMultipartProvider .cs:L1-61</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> Please hide all details of implementation by interface. SOLID...Class should be acceptable for extension not for MODIFICATION! The second part you should not rely on implementation of MultipartFileStreamProvider it can be changed...\n- <a href='https://github.com/IrynaRobulets'><img border=0 src='https://avatars.githubusercontent.com/u/17322068?v=3' height=16 width=16'></a> If I use interface instead of AzureBlobStorageMultipartProvider. I can't use ReadAsMultipartAsync<AzureBlobStorageMultipartProvider>(_multipartStreamProvider) becuase this method requires paramater refernces to abstract class MultipartStreamProvider. As a result I need to make my own conten reading provider. Am I understand you?\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> Interesting case. But why you did't accept parameters into action body? Instear using Request property of ApiController? As I understand Web Api allows to have mixed parameters inside action signature.\n- [ ] <a href='#crh-comment-Pull 014860779d7aa5040ca650c94cf285d073d7caa6 DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Azure/AzureBlobStorageMultipartProvider%20.cs 15'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r62654602'>File: DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Azure/AzureBlobStorageMultipartProvider .cs:L1-61</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> Nice injection...Why you are not using the same approach with MultipartProvider?\n- <a href='https://github.com/IrynaRobulets'><img border=0 src='https://avatars.githubusercontent.com/u/17322068?v=3' height=16 width=16'></a> Do you mean MultipartFileStreamProvider? Why I need to inject it, there is no place where I create an instance. It's only inheritance from this class.\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> Yep, need to think how to better use this Provider.\n- [ ] <a href='#crh-comment-Pull 014860779d7aa5040ca650c94cf285d073d7caa6 DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs 23'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r62654694'>File: DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs:L1-77</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> You really need expose access to all persictence layer?\n- <a href='https://github.com/IrynaRobulets'><img border=0 src='https://avatars.githubusercontent.com/u/17322068?v=3' height=16 width=16'></a> Nope, I've changed it\n- [ ] <a href='#crh-comment-Pull 014860779d7aa5040ca650c94cf285d073d7caa6 DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs 45'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r62654898'>File: DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs:L1-77</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> Please don't mix persistence layer with Azure complexity...It should be separarted btw DbContext also support async operations.\n- [ ] <a href='#crh-comment-Pull 014860779d7aa5040ca650c94cf285d073d7caa6 DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Extensions/HttpResponseExtensions.cs 12'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r62654982'>File: DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Extensions/HttpResponseExtensions.cs:L1-19</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> Can you explain why this method is not extension method with a single responsibility?\n- <a href='https://github.com/IrynaRobulets'><img border=0 src='https://avatars.githubusercontent.com/u/17322068?v=3' height=16 width=16'></a> Now it's with single responsibility.\n- [ ] <a href='#crh-comment-Pull 014860779d7aa5040ca650c94cf285d073d7caa6 DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Package/PreferencesPackage.cs 34'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r62655310'>File: DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Package/PreferencesPackage.cs:L1-43</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> You really need this registration?\n- <a href='https://github.com/IrynaRobulets'><img border=0 src='https://avatars.githubusercontent.com/u/17322068?v=3' height=16 width=16'></a> I'm not sure exectly, I guess if user not null he has permissions to update photo. I check permissions because they can be changed in the azure out of code. So, if supose that permissions won't be changed in Azure this checking have no sence. Do I need get it away?\n- [ ] <a href='#crh-comment-Pull c2f6558c0b2f7c4aa483ff4f76aeb0a17dbb3403 DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Azure/AzureBlobStorageMultipartProvider%20.cs 44'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r62712554'>File: DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Azure/AzureBlobStorageMultipartProvider .cs:L1-61</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> You can return file location from method UpdateBlob. It's covered by case when after uploading file you will get path to it location.\n- [ ] <a href='#crh-comment-Pull 0c48539fac6fe2fe3aff2df30f155f5bfaea9cca DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/AvatarOperations.cs 16'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r63206150'>File: DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/AvatarOperations.cs:L1-35</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> Method with a Get prefix but inside the body it;s call UpdateBlob? Can you clarify this....\n- [ ] <a href='#crh-comment-Pull 0c48539fac6fe2fe3aff2df30f155f5bfaea9cca DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Queries/PreferencesQuery.cs 19'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r63206731'>File: DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Queries/PreferencesQuery.cs:L1-36</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> I see a small smell...You are exposing Users property and also declare GetUser method in interface? Do not many methods inside interface? Where our flexibility related to IOrderedQueryable?\n- [ ] <a href='#crh-comment-Pull 0c48539fac6fe2fe3aff2df30f155f5bfaea9cca DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Queries/PreferencesQuery.cs 31'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r63207315'>File: DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Queries/PreferencesQuery.cs:L1-36</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> May be it's better to pass User entity with updated state? Because now I don't see any reson to pass this additional parameter as URL.\n- [ ] <a href='#crh-comment-Pull 0c48539fac6fe2fe3aff2df30f155f5bfaea9cca DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Models/FileModel.cs 11'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r63207901'>File: DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Models/FileModel.cs:L1-29</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> We need this alone constructor?\n- [ ] <a href='#crh-comment-Pull 0c48539fac6fe2fe3aff2df30f155f5bfaea9cca DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Formatters/FileMediaFormatter.cs 57'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r63208190'>File: DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Formatters/FileMediaFormatter.cs:L1-86</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> Move to extesion methods.\n- [ ] <a href='#crh-comment-Pull 0c48539fac6fe2fe3aff2df30f155f5bfaea9cca DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Formatters/FileMediaFormatter.cs 70'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r63208445'>File: DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Formatters/FileMediaFormatter.cs:L1-86</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> Can this method simplified?\n- [ ] <a href='#crh-comment-Pull 0c48539fac6fe2fe3aff2df30f155f5bfaea9cca DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/AvatarOperations.cs 25'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r63208716'>File: DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/AvatarOperations.cs:L1-35</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> May be you should first check is fomat valid and the setting up all properties?\n- <a href='https://github.com/IrynaRobulets'><img border=0 src='https://avatars.githubusercontent.com/u/17322068?v=3' height=16 width=16'></a> Yeap, it's more logical\n- [ ] <a href='#crh-comment-Pull 0c48539fac6fe2fe3aff2df30f155f5bfaea9cca DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs 44'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r63208874'>File: DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs:L1-81</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> Please rename to something like IsUrlValidbecause I don't see that we work with actual files. Only urls. Correct?\n- <a href='https://github.com/IrynaRobulets'><img border=0 src='https://avatars.githubusercontent.com/u/17322068?v=3' height=16 width=16'></a> 1. Done\n2. I guess that method name is not the best. The idea is to check if the avatar url have changed than we need to update this field in database and delete unnecessary file from blobstorage. If url the same, there is no point to update url in database. Maybe the method name like IsUrlSimilar will be more clear.\n- [ ] <a href='#crh-comment-Pull 0c48539fac6fe2fe3aff2df30f155f5bfaea9cca DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs 74'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r63209219'>File: DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs:L1-81</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> Btw it ca be moved to UserExtension method to chechking url validity.\n- [ ] <a href='#crh-comment-Pull 0c48539fac6fe2fe3aff2df30f155f5bfaea9cca DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Extensions/HttpResponseExtensions.cs 15'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r63209255'>File: DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Extensions/HttpResponseExtensions.cs:L1-20</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> I like such small and concrete extension methods.\n- [ ] <a href='#crh-comment-Pull 0c48539fac6fe2fe3aff2df30f155f5bfaea9cca DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs 35'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/ITsvetkoFF/Kv-013/pull/138#discussion_r63209487'>File: DevelopGitHub/GitHubExtension/GitHubExtention.Preferences.WebApi/Controllers/UserPreferencesController.cs:L1-81</a></b>\n- <a href='https://github.com/TerraVenil'><img border=0 src='https://avatars.githubusercontent.com/u/6068068?v=3' height=16 width=16'></a> So I suggest all magic happens inside formatter and here you already have filled all necessesary fields?\n- <a href='https://github.com/IrynaRobulets'><img border=0 src='https://avatars.githubusercontent.com/u/17322068?v=3' height=16 width=16'></a> Yes\n\n\n<a href='https://www.codereviewhub.com/ITsvetkoFF/Kv-013/pull/138?mark_as_completed=1'><img src='http://www.codereviewhub.com/site/github-mark-as-completed.png' height=26></a>&nbsp;<a href='https://www.codereviewhub.com/ITsvetkoFF/Kv-013/pull/138?approve=1'><img src='http://www.codereviewhub.com/site/github-approve.png' height=26></a>&nbsp;<a href='https://github.com/ITsvetkoFF/Kv-013/pull/138'><img src='http://www.codereviewhub.com/site/github-refresh.png' height=26></a>\n<a href='#crh-end'></a>","created_at":"2016-05-06T17:03:31Z","updated_at":"2016-05-13T17:25:42Z","closed_at":null,"merged_at":null,"merge_commit_sha":"489861a7f16cc359d1b3575a0f632d999cfba594","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/pulls/138/commits","review_comments_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/pulls/138/comments","review_comment_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/pulls/comments{/number}","comments_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/issues/138/comments","statuses_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/statuses/0c48539fac6fe2fe3aff2df30f155f5bfaea9cca","head":{"label":"ITsvetkoFF:feature/upload-photo-backend","ref":"feature/upload-photo-backend","sha":"0c48539fac6fe2fe3aff2df30f155f5bfaea9cca","user":{"login":"ITsvetkoFF","id":7273003,"avatar_url":"https://avatars.githubusercontent.com/u/7273003?v=3","gravatar_id":"","url":"https://api.github.com/users/ITsvetkoFF","html_url":"https://github.com/ITsvetkoFF","followers_url":"https://api.github.com/users/ITsvetkoFF/followers","following_url":"https://api.github.com/users/ITsvetkoFF/following{/other_user}","gists_url":"https://api.github.com/users/ITsvetkoFF/gists{/gist_id}","starred_url":"https://api.github.com/users/ITsvetkoFF/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ITsvetkoFF/subscriptions","organizations_url":"https://api.github.com/users/ITsvetkoFF/orgs","repos_url":"https://api.github.com/users/ITsvetkoFF/repos","events_url":"https://api.github.com/users/ITsvetkoFF/events{/privacy}","received_events_url":"https://api.github.com/users/ITsvetkoFF/received_events","type":"User","site_admin":false},"repo":{"id":52432402,"name":"Kv-013","full_name":"ITsvetkoFF/Kv-013","owner":{"login":"ITsvetkoFF","id":7273003,"avatar_url":"https://avatars.githubusercontent.com/u/7273003?v=3","gravatar_id":"","url":"https://api.github.com/users/ITsvetkoFF","html_url":"https://github.com/ITsvetkoFF","followers_url":"https://api.github.com/users/ITsvetkoFF/followers","following_url":"https://api.github.com/users/ITsvetkoFF/following{/other_user}","gists_url":"https://api.github.com/users/ITsvetkoFF/gists{/gist_id}","starred_url":"https://api.github.com/users/ITsvetkoFF/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ITsvetkoFF/subscriptions","organizations_url":"https://api.github.com/users/ITsvetkoFF/orgs","repos_url":"https://api.github.com/users/ITsvetkoFF/repos","events_url":"https://api.github.com/users/ITsvetkoFF/events{/privacy}","received_events_url":"https://api.github.com/users/ITsvetkoFF/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/ITsvetkoFF/Kv-013","description":"","fork":false,"url":"https://api.github.com/repos/ITsvetkoFF/Kv-013","forks_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/forks","keys_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/keys{/key_id}","collaborators_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/teams","hooks_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/hooks","issue_events_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/issues/events{/number}","events_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/events","assignees_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/assignees{/user}","branches_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/branches{/branch}","tags_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/tags","blobs_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/git/refs{/sha}","trees_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/git/trees{/sha}","statuses_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/statuses/{sha}","languages_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/languages","stargazers_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/stargazers","contributors_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/contributors","subscribers_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/subscribers","subscription_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/subscription","commits_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/commits{/sha}","git_commits_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/git/commits{/sha}","comments_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/comments{/number}","issue_comment_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/issues/comments{/number}","contents_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/contents/{+path}","compare_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/compare/{base}...{head}","merges_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/merges","archive_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/downloads","issues_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/issues{/number}","pulls_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/pulls{/number}","milestones_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/milestones{/number}","notifications_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/labels{/name}","releases_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/releases{/id}","deployments_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/deployments","created_at":"2016-02-24T10:00:53Z","updated_at":"2016-05-09T10:52:56Z","pushed_at":"2016-05-13T16:45:55Z","git_url":"git://github.com/ITsvetkoFF/Kv-013.git","ssh_url":"git@github.com:ITsvetkoFF/Kv-013.git","clone_url":"https://github.com/ITsvetkoFF/Kv-013.git","svn_url":"https://github.com/ITsvetkoFF/Kv-013","homepage":null,"size":3725,"stargazers_count":7,"watchers_count":7,"language":"C#","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":22,"forks":0,"open_issues":22,"watchers":7,"default_branch":"develop"}},"base":{"label":"ITsvetkoFF:develop","ref":"develop","sha":"c7631eaedc92dcc88948facefa042c41eda840d8","user":{"login":"ITsvetkoFF","id":7273003,"avatar_url":"https://avatars.githubusercontent.com/u/7273003?v=3","gravatar_id":"","url":"https://api.github.com/users/ITsvetkoFF","html_url":"https://github.com/ITsvetkoFF","followers_url":"https://api.github.com/users/ITsvetkoFF/followers","following_url":"https://api.github.com/users/ITsvetkoFF/following{/other_user}","gists_url":"https://api.github.com/users/ITsvetkoFF/gists{/gist_id}","starred_url":"https://api.github.com/users/ITsvetkoFF/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ITsvetkoFF/subscriptions","organizations_url":"https://api.github.com/users/ITsvetkoFF/orgs","repos_url":"https://api.github.com/users/ITsvetkoFF/repos","events_url":"https://api.github.com/users/ITsvetkoFF/events{/privacy}","received_events_url":"https://api.github.com/users/ITsvetkoFF/received_events","type":"User","site_admin":false},"repo":{"id":52432402,"name":"Kv-013","full_name":"ITsvetkoFF/Kv-013","owner":{"login":"ITsvetkoFF","id":7273003,"avatar_url":"https://avatars.githubusercontent.com/u/7273003?v=3","gravatar_id":"","url":"https://api.github.com/users/ITsvetkoFF","html_url":"https://github.com/ITsvetkoFF","followers_url":"https://api.github.com/users/ITsvetkoFF/followers","following_url":"https://api.github.com/users/ITsvetkoFF/following{/other_user}","gists_url":"https://api.github.com/users/ITsvetkoFF/gists{/gist_id}","starred_url":"https://api.github.com/users/ITsvetkoFF/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ITsvetkoFF/subscriptions","organizations_url":"https://api.github.com/users/ITsvetkoFF/orgs","repos_url":"https://api.github.com/users/ITsvetkoFF/repos","events_url":"https://api.github.com/users/ITsvetkoFF/events{/privacy}","received_events_url":"https://api.github.com/users/ITsvetkoFF/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/ITsvetkoFF/Kv-013","description":"","fork":false,"url":"https://api.github.com/repos/ITsvetkoFF/Kv-013","forks_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/forks","keys_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/keys{/key_id}","collaborators_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/teams","hooks_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/hooks","issue_events_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/issues/events{/number}","events_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/events","assignees_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/assignees{/user}","branches_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/branches{/branch}","tags_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/tags","blobs_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/git/refs{/sha}","trees_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/git/trees{/sha}","statuses_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/statuses/{sha}","languages_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/languages","stargazers_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/stargazers","contributors_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/contributors","subscribers_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/subscribers","subscription_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/subscription","commits_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/commits{/sha}","git_commits_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/git/commits{/sha}","comments_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/comments{/number}","issue_comment_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/issues/comments{/number}","contents_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/contents/{+path}","compare_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/compare/{base}...{head}","merges_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/merges","archive_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/downloads","issues_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/issues{/number}","pulls_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/pulls{/number}","milestones_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/milestones{/number}","notifications_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/labels{/name}","releases_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/releases{/id}","deployments_url":"https://api.github.com/repos/ITsvetkoFF/Kv-013/deployments","created_at":"2016-02-24T10:00:53Z","updated_at":"2016-05-09T10:52:56Z","pushed_at":"2016-05-13T16:45:55Z","git_url":"git://github.com/ITsvetkoFF/Kv-013.git","ssh_url":"git@github.com:ITsvetkoFF/Kv-013.git","clone_url":"https://github.com/ITsvetkoFF/Kv-013.git","svn_url":"https://github.com/ITsvetkoFF/Kv-013","homepage":null,"size":3725,"stargazers_count":7,"watchers_count":7,"language":"C#","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":22,"forks":0,"open_issues":22,"watchers":7,"default_branch":"develop"}},"_links":{"self":{"href":"https://api.github.com/repos/ITsvetkoFF/Kv-013/pulls/138"},"html":{"href":"https://github.com/ITsvetkoFF/Kv-013/pull/138"},"issue":{"href":"https://api.github.com/repos/ITsvetkoFF/Kv-013/issues/138"},"comments":{"href":"https://api.github.com/repos/ITsvetkoFF/Kv-013/issues/138/comments"},"review_comments":{"href":"https://api.github.com/repos/ITsvetkoFF/Kv-013/pulls/138/comments"},"review_comment":{"href":"https://api.github.com/repos/ITsvetkoFF/Kv-013/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/ITsvetkoFF/Kv-013/pulls/138/commits"},"statuses":{"href":"https://api.github.com/repos/ITsvetkoFF/Kv-013/statuses/0c48539fac6fe2fe3aff2df30f155f5bfaea9cca"}}}}