Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Timeline API: Exception: {"Requested value 'reviewed' was not found."} #1525

Closed
aniket-aurea opened this issue Dec 28, 2016 · 3 comments · Fixed by #1563
Closed

Timeline API: Exception: {"Requested value 'reviewed' was not found."} #1525

aniket-aurea opened this issue Dec 28, 2016 · 3 comments · Fixed by #1563
Labels
Type: Bug Something isn't working as documented

Comments

@aniket-aurea
Copy link

aniket-aurea commented Dec 28, 2016

Note: gitHubClient.PullRequest.Get works fine for the same pull request
StackTrace:
at System.Enum.EnumResult.SetFailure(ParseFailureKind failure, String failureMessageID, Object failureMessageFormatArgument)
at System.Enum.TryParseEnum(Type enumType, String value, Boolean ignoreCase, EnumResult& parseResult)
at System.Enum.Parse(Type enumType, String value, Boolean ignoreCase)
at Octokit.Internal.SimpleJsonSerializer.GitHubSerializerStrategy.DeserializeObject(Object value, Type type) in C:\Users\shiftkey\code\octokit.net\Octokit\Http\SimpleJsonSerializer.cs:line 125
at Octokit.PocoJsonSerializerStrategy.DeserializeObject(Object value, Type type) in C:\Users\shiftkey\code\octokit.net\Octokit\SimpleJson.cs:line 1478
at Octokit.Internal.SimpleJsonSerializer.GitHubSerializerStrategy.DeserializeObject(Object value, Type type) in C:\Users\shiftkey\code\octokit.net\Octokit\Http\SimpleJsonSerializer.cs:line 166
at Octokit.PocoJsonSerializerStrategy.DeserializeObject(Object value, Type type) in C:\Users\shiftkey\code\octokit.net\Octokit\SimpleJson.cs:line 1505
at Octokit.Internal.SimpleJsonSerializer.GitHubSerializerStrategy.DeserializeObject(Object value, Type type) in C:\Users\shiftkey\code\octokit.net\Octokit\Http\SimpleJsonSerializer.cs:line 166
at Octokit.SimpleJson.DeserializeObject(String json, Type type, IJsonSerializerStrategy jsonSerializerStrategy) in C:\Users\shiftkey\code\octokit.net\Octokit\SimpleJson.cs:line 583
at Octokit.SimpleJson.DeserializeObject[T](String json, IJsonSerializerStrategy jsonSerializerStrategy) in C:\Users\shiftkey\code\octokit.net\Octokit\SimpleJson.cs:line 595
at Octokit.Internal.SimpleJsonSerializer.Deserialize[T](String json) in C:\Users\shiftkey\code\octokit.net\Octokit\Http\SimpleJsonSerializer.cs:line 21
at Octokit.Internal.JsonHttpPipeline.DeserializeResponse[T](IResponse response) in C:\Users\shiftkey\code\octokit.net\Octokit\Http\JsonHttpPipeline.cs:line 62
at Octokit.Connection.d__531.MoveNext() in C:\Users\shiftkey\code\octokit.net\Octokit\Http\Connection.cs:line 560 --- End of stack trace from previous location where exception was thrown --- at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task) at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) at Octokit.ApiConnection.<GetPage>d__401.MoveNext() in C:\Users\shiftkey\code\octokit.net\Octokit\Http\ApiConnection.cs:line 0
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.ApiConnection.<>c__DisplayClass17_01.<<GetAll>b__0>d.MoveNext() in C:\Users\shiftkey\code\octokit.net\Octokit\Http\ApiConnection.cs:line 0 --- End of stack trace from previous location where exception was thrown --- at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task) at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) at Octokit.ApiPagination.<GetAllPages>d__01.MoveNext() in C:\Users\shiftkey\code\octokit.net\Octokit\Clients\ApiPagination.cs:line 0

@shiftkey
Copy link
Member

shiftkey commented Jan 9, 2017

@aniket-aurea do you have an example URL that I can use to confirm this is related to EventInfoState not having this value?

@shiftkey shiftkey added the bug label Jan 9, 2017
@ryangribble
Copy link
Contributor

ryangribble commented Jan 9, 2017

Apart from adding enum values that are missing and waiting for another release, we also have a proposal to handle these at runtime here #1504

@aniket-aurea
Copy link
Author

aniket-aurea commented Jan 9, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Something isn't working as documented
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants