Skip to content

chore(gw): extract logical functions to improve readability - #8885

Merged
Jorropo merged 3 commits into
ipfs:masterfrom
justindotpub:justincjohnson/refactor-gateway
Apr 15, 2022
Merged

Jorropo merged 3 commits into
ipfs:masterfrom
justindotpub:justincjohnson/refactor-gateway

Conversation

@justindotpub

Copy link
Copy Markdown
Contributor

Another take at #8883 from my personal account this time.

@Jorropo Jorropo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is definitely clearer. thx 🙂

Just a few final things.

Comment thread core/corehttp/gateway_handler.go Outdated
Comment thread core/corehttp/gateway_handler.go Outdated
Comment thread core/corehttp/gateway_handler.go Outdated
Comment thread core/corehttp/gateway_handler.go Outdated
Comment thread core/corehttp/gateway_handler.go Outdated
Comment thread core/corehttp/gateway_handler.go Outdated
@justindotpub

Copy link
Copy Markdown
Contributor Author

I believe I've addressed everything @Jorropo. Let me know if I missed anything, and thanks again for the feedback.

@Jorropo Jorropo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 🙏

@Jorropo
Jorropo merged commit e07baf5 into ipfs:master Apr 15, 2022
@justindotpub
justindotpub deleted the justincjohnson/refactor-gateway branch April 15, 2022 14:10
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