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

System.ArgumentException: Value 'transferred' is not a valid 'EventInfoState' enum value. #2023

Closed
terrajobst opened this issue Oct 3, 2019 · 4 comments · Fixed by #2024
Closed
Assignees

Comments

@terrajobst
Copy link

I'm calling EventInfo.Event.Value, which apparently doesn't work for issues that were transferred:

Unhandled exception. System.ArgumentException: Value 'transferred' is not a valid 'EventInfoState' enum value.
   at Octokit.StringEnum`1.ParseValue() in C:\Users\shiftkey\src\octokit.net\Octokit\Models\Response\StringEnum.cs:line 146
   at Octokit.StringEnum`1.get_Value() in C:\Users\shiftkey\src\octokit.net\Octokit\Models\Response\StringEnum.cs:line 45
   at ApiReviewList.Reports.ApiReviewFeedback.<GetAsync>g__IsApiEvent|1_1(EventInfo eventInfo) in P:\api-review-list\create-past-reviews\Reports\ApiReviewFeedback.cs:line 42
   at ApiReviewList.Reports.ApiReviewFeedback.<>c__DisplayClass1_0.<GetAsync>b__10(EventInfo e) in P:\api-review-list\create-past-reviews\Reports\ApiReviewFeedback.cs:line 57
   at System.Linq.Enumerable.WhereEnumerableIterator`1.MoveNext()
   at System.Linq.Lookup`2.Create(IEnumerable`1 source, Func`2 keySelector, IEqualityComparer`1 comparer)
   at System.Linq.GroupedEnumerable`2.GetEnumerator()
   at ApiReviewList.Reports.ApiReviewFeedback.<GetAsync>g__GetApiEvents|1_2(IEnumerable`1 events, SortedSet`1 dateSet)+MoveNext() in P:\api-review-list\create-past-reviews\Reports\ApiReviewFeedback.cs:line 57
   at ApiReviewList.Reports.ApiReviewFeedback.GetAsync(IEnumerable`1 dates) in P:\api-review-list\create-past-reviews\Reports\ApiReviewFeedback.cs:line 139
   at ApiReviewList.Reports.ApiReviewSummary.GetAsync(IEnumerable`1 dates) in P:\api-review-list\create-past-reviews\Reports\ApiReviewSummary.cs:line 25
   at create_past_reviews.Program.Main(String[] args) in P:\api-review-list\create-past-reviews\Program.cs:line 20
   at create_past_reviews.Program.<Main>(String[] args)
@shiftkey
Copy link
Member

shiftkey commented Oct 3, 2019

@terrajobst probably not going to be able to look at this until the weekend, but if you can get a reproducible failure I can run myself it'll save me a lot of time to verify the fix.

@shiftkey
Copy link
Member

shiftkey commented Oct 3, 2019

public enum EventInfoState

This is the type that probably needs a new transferred state

@shiftkey
Copy link
Member

shiftkey commented Oct 3, 2019

@terrajobst to nerd-snipe you into helping me out a bit, if you wanted to open a PR with that new field it'll generate a new NuGet package on the beta feed which you can then install and verify the fix on your end.

@hnrkndrssn
Copy link
Contributor

hnrkndrssn commented Oct 3, 2019

If no one gets to this, I can get a PR together tonight.

Side note: the Events docs doesn't include a transferred event type...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants