-
Notifications
You must be signed in to change notification settings - Fork 35
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
[BUG]: app.eachInstallation Type Doesn't include Account Details #465
Comments
👋 Hi! Thank you for this contribution! Just to let you know, our GitHub SDK team does a round of issue and PR reviews twice a week, every Monday and Friday! We have a process in place for prioritizing and responding to your input. Because you are a part of this community please feel free to comment, add to, or pick up any issues/PRs that are labled with |
This seems like an issue from GitHub's side from their OpenAPI spec. This seems like the same as octokit/openapi-types.ts#339 |
Can you upgrade to v13.1.8 as there has been some accidental breaking changes that were released in previous versions. Is there a particular reason you aren't upgrading to v14? |
@wolfy1339 ah, we figured it was something with the types. We run into this error on v13.1.8. We are trying to use v14 but we have the same issue. We currently put a workaround in to parse the data we need from an HTML URL instead using v14. |
What happened?
When upgrading from v13.1.1 to v13.1.3 we have seen issues using data with
app.eachInstallation
.This is an example of our code that breaks when upgrading to anything past v13.1.1.
On v13.1.1 it works, but, on v13.1.2 and beyond, we get this type error.
Versions
Node: v18.14.0
Relevant log output
No response
Code of Conduct
The text was updated successfully, but these errors were encountered: