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

fix: remove axios as runtime dependency #719

Merged
merged 2 commits into from
Dec 15, 2023
Merged

fix: remove axios as runtime dependency #719

merged 2 commits into from
Dec 15, 2023

Conversation

pgrzy-golem
Copy link
Contributor

JST-613

README.md Outdated
@@ -241,3 +241,5 @@ yarn format
- Learn about preparing your own Docker-like images for
the [VM runtime](https://docs.golem.network/docs/creators/javascript/examples/tools/converting-docker-image-to-golem-format)
- Write your own app with [JavaScript API](https://docs.golem.network/docs/creators/javascript/quickstarts/quickstart)

## How to
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like added by mistake. I'm going to remove this.

@@ -178,7 +178,7 @@ export class PaymentService {
}

private async getPaymentAddress(): Promise<string> {
const { data } = await this.yagnaApi.identity.getIdentity();
const data = await this.yagnaApi.identity.getIdentity();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Check with regression tests

@grisha87 grisha87 merged commit 9c8fc52 into beta Dec 15, 2023
15 of 16 checks passed
@grisha87 grisha87 deleted the feature/JST-613 branch December 15, 2023 13:07
@grisha87 grisha87 restored the feature/JST-613 branch December 15, 2023 13:07
@grisha87 grisha87 deleted the feature/JST-613 branch December 15, 2023 13:07
Copy link

🎉 This PR is included in version 2.0.0-beta.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version 2.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

2 participants