Skip to content
This repository has been archived by the owner on Jun 12, 2023. It is now read-only.

fix: flask1.25 #246

Merged
merged 7 commits into from
Mar 3, 2023
Merged

fix: flask1.25 #246

merged 7 commits into from
Mar 3, 2023

Conversation

irubido
Copy link
Contributor

@irubido irubido commented Feb 28, 2023

Fixed filsnap for flask v1.25

@irubido irubido requested a review from Lykhoyda February 28, 2023 13:24
@github-actions github-actions bot temporarily deployed to Preview: (irubido/flask1.25) February 28, 2023 16:24 Inactive
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 28, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3f43119
Status: ✅  Deploy successful!
Preview URL: https://47b68acf.filsnap.pages.dev
Branch Preview URL: https://irubido-flask1-25.filsnap.pages.dev

View logs

@github-actions github-actions bot temporarily deployed to Preview: (irubido/flask1.25) March 2, 2023 12:52 Inactive
@github-actions github-actions bot temporarily deployed to Preview: (irubido/flask1.25) March 2, 2023 13:04 Inactive
@github-actions github-actions bot temporarily deployed to Preview: (irubido/flask1.25) March 2, 2023 13:06 Inactive
@irubido irubido marked this pull request as ready for review March 2, 2023 13:08
@github-actions github-actions bot temporarily deployed to Preview: (irubido/flask1.25) March 2, 2023 13:12 Inactive
Copy link
Contributor

@BeroBurny BeroBurny left a comment

Choose a reason for hiding this comment

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

please remove walletStub.rpcStubs.web3_clientVersion from packages/snap/test/unit/filecoin/account.test.ts

packages/adapter/src/methods.ts Outdated Show resolved Hide resolved
packages/snap/test/unit/filecoin/message.test.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@BeroBurny BeroBurny left a comment

Choose a reason for hiding this comment

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

LGTM

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

Successfully merging this pull request may close these issues.

2 participants