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

Declare Groups API allProjects function default type first (#3624) #3624

Merged
merged 1 commit into from
Sep 14, 2024

Conversation

ArnaudTA
Copy link
Contributor

@ArnaudTA ArnaudTA commented Sep 9, 2024

Hello in you code the return of allProjects is CondensedProjectSchema[].
Apparently for ts if no options are provided it matches the first signature.
So I just interved the two signatures to be closer to the api default behavior

ps: topics can be null in your schema but i don't see anything on it in the documentation. Feel free to edit my pr to remove it if you think it is a good idea.

@jdalrymple jdalrymple added the type:types Adding or enhancing typescript types label Sep 14, 2024
@jdalrymple jdalrymple merged commit ee74d37 into jdalrymple:main Sep 14, 2024
2 checks passed
@jdalrymple jdalrymple changed the title refactor: 🏷️ declare default type first Declare Groups API allProjects function default type first (#3624) Sep 14, 2024
@jdalrymple
Copy link
Owner

🚀 PR was released in 40.2.1 🚀

@jdalrymple jdalrymple added the released This issue/pull request has been released. label Sep 14, 2024
@ArnaudTA ArnaudTA deleted the fix/all-projects-types branch September 14, 2024 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released This issue/pull request has been released. type:types Adding or enhancing typescript types
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants