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

feat: Add missing type exports #66

Closed
wants to merge 2 commits into from

Conversation

ieedan
Copy link

@ieedan ieedan commented Sep 21, 2024

Exported any not yet exported user facing types.

I also exported SwapiApi just cause that is what shows up in the function signature. I think this makes it less confusing... If we want to just export one or the other we could change the function signature to use Swapy instead? Let me know your thoughts!

  • Build to dist

@ieedan ieedan changed the title feat: Add missing exports feat: Add missing type exports Sep 21, 2024
@TahaSh
Copy link
Owner

TahaSh commented Sep 21, 2024

@ieedan Thanks for this PR! When you submitted it, I was already working on exporting event types. I've also replaced SwapyApi with Swapy interface. Thank you again 🙏

@TahaSh TahaSh closed this Sep 21, 2024
@ieedan ieedan deleted the add-missing-type-exports branch September 21, 2024 15:50
@ieedan
Copy link
Author

ieedan commented Sep 21, 2024

Sorry about that I only found the time this morning! Thanks!

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 this pull request may close these issues.

feat: Export missing type definitions
2 participants