Skip to content
This repository was archived by the owner on Oct 11, 2024. It is now read-only.

Conversation

@kimpers
Copy link
Contributor

@kimpers kimpers commented Oct 12, 2021

No description provided.

@vercel
Copy link

vercel bot commented Oct 12, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/0x-eng/website/GFCH1G78fAgAqKLR9AQzb7WMUndv
✅ Preview: https://website-git-kim-fixeip1559-gas-estimation-0x-eng.vercel.app

key: fs.readFileSync('./server.key'),
cert: fs.readFileSync('./server.cert'),
};
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

With SSL turned on for local dev the page won't even load in Chrome due to an invalid cert 🙈

Copy link
Member

Choose a reason for hiding this comment

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

Is this because the cert is expired?

Copy link
Member

Choose a reason for hiding this comment

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

I just checked. I didn't see the ./server.key or ./server.cert exist.

Copy link
Contributor Author

@kimpers kimpers Oct 13, 2021

Choose a reason for hiding this comment

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

yeah I don't have it either 😂 Previously you could just ignore the invalid config in Chrome but not anymore. I don't think https for local dev adds much value so might as well remove it 🙈

Copy link
Member

Choose a reason for hiding this comment

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

Cool. Please let me know if you think you need it in the future, I think I can do something about it.

"lint": "tslint --format stylish --project . 'ts/**/*.ts' 'ts/**/*.tsx'",
"fix": "yarn lint --fix",
"pre_push": "yarn lint:prettier && yarn lint && yarn test",
"pre_push": "yarn typecheck && yarn lint:prettier && yarn lint && yarn test",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

actually checking types unveiled a few lingering TS type issues:feelsgood:

Copy link
Contributor

@johnrjj johnrjj left a comment

Choose a reason for hiding this comment

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

LGTM

@kimpers kimpers merged commit 1c10e93 into development Oct 13, 2021
@kimpers kimpers deleted the kim/fix/eip1559-gas-estimation branch October 13, 2021 15:23
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.

4 participants