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 rinkeby which is not used anymore #24

Merged
merged 2 commits into from
Oct 13, 2022
Merged

Conversation

fgiannotti
Copy link
Owner

Por el tema del 'The merge' con ethereum. Rinkeby está deprecada e infura no nos la deja usar mas, hay que usar goerli o sepolia.
Busque unos faucets y me dí tokens a la wallet 'nuestra' de metamask (la que está en las envs) e hice un par de transacciones y funcionaron.

Call.js funcionando:

image

Tmb desde el web server se arreglo

image

Comment on lines +9 to +15
import fetch from 'node-fetch';
import { abortableFetch } from 'abortcontroller-polyfill/dist/cjs-ponyfill';
// To fix: https://github.com/node-fetch/node-fetch/issues/784
// useful: https://www.npmjs.com/package/node-abort-controller, but I used this: https://lightrun.com/answers/apollographql-apollo-client-expected-signal-to-be-an-instanceof-abortsignal
global.fetch = abortableFetch(fetch).fetch;

declare const module: any;
Copy link
Owner Author

@fgiannotti fgiannotti Oct 10, 2022

Choose a reason for hiding this comment

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

Esto arregla un problema mío que me saltaba con un bug que tiene node-fetch. (node-fetch/node-fetch#784)
Algo de CONNECTION ERROR no se que, y por detrás debugeando muuucho encontre otro error escondido que es exactamente el que está en el issue) googleando un toque varios fix este me funcionó.

A todo esto Node-fetch lo solucionó ya. Pero no tenemos esa versión bajada (Y no podemos subirla porque es una dependencia que usa web3, no nosotros directamente)

@SantiagoAspres SantiagoAspres requested review from SantiagoAspres and removed request for SantiagoAspres October 11, 2022 02:04
@fgiannotti fgiannotti merged commit c6ccda6 into master Oct 13, 2022
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.

2 participants