Conversation
NSString *previewContentType = [NSString stringWithFormat:@"application/vnd.github.%@+json", OCTClientPreviewAPIVersion]; | ||
NSString *baseContentType = @"application/vnd.github.%@+json"; | ||
NSString *stableContentType = [NSString stringWithFormat:baseContentType, OCTClientAPIVersion]; | ||
NSString *previewContentType = [NSString stringWithFormat:baseContentType, OCTClientPreviewAPIVersion]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we still need this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't look like it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We do, apparently still in preview.
@@ -16,7 +16,7 @@ | |||
@implementation OCTClient (Organizations) | |||
|
|||
- (RACSignal *)fetchUserOrganizations { | |||
return [[self enqueueUserRequestWithMethod:@"GET" relativePath:@"/orgs" parameters:nil resultClass:OCTOrganization.class] oct_parsedResults]; | |||
return [[self enqueueUserRequestWithMethod:@"GET" relativePath:@"/user/orgs" parameters:nil resultClass:OCTOrganization.class] oct_parsedResults]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm getting 404s, because “user requests” already have this prefix.
🎱 |
@@ -44,8 +44,13 @@ | |||
NSString * const OCTClientErrorMessagesKey = @"OCTClientErrorMessagesKey"; | |||
|
|||
NSString * const OCTClientAPIVersion = @"v3"; | |||
|
|||
/// See https://developer.github.com/changes/2014-12-08-removing-authorizations-token/ | |||
NSString * const OCTClientPreviewAPIVersion = @"mirage-preview"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you rename this to clarify which preview API it's talking about?
9️⃣ |
0️⃣ |
API changes in light of https://developer.github.com/changes/2014-12-08-organization-permissions-api-preview/