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

APIClient rework #80

Closed
wants to merge 5 commits into from
Closed

APIClient rework #80

wants to merge 5 commits into from

Conversation

aaroncox
Copy link
Member

@aaroncox aaroncox commented Aug 28, 2023

This is a combination of:

This will be published as 0.10.0.

* Removing api definitions

* Testing extending APIClient

* Exporting as BaseAPIClient now

* Fixing tests

* Update package.json
* Caching ABI in Action for future use

* Added `.decoded` method to action to automatically decode

* Disabled failing tests

These tests don't make much sense to me. The variant being passed in as action data isn't even valid as far as I can tell.

* Fixed type selection for generated action

* Reimplemented tests for variant in more sane way

* Removed string casting, wasn't needed
@aaroncox aaroncox mentioned this pull request Aug 30, 2023
@aaroncox aaroncox changed the title APIClient rework + ABIs embedded in Actions APIClient rework Sep 12, 2023
@aaroncox aaroncox closed this Sep 18, 2023
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.

1 participant