Skip to content

Conversation

@rg911
Copy link
Contributor

@rg911 rg911 commented Oct 3, 2020

Updated packages.json

@rg911 rg911 requested a review from fboucquez October 3, 2020 14:47
@fboucquez
Copy link
Contributor

Do we need the "request" packages @rg911? Should fetch be enough? What happens if we remove request?

@rg911
Copy link
Contributor Author

rg911 commented Oct 5, 2020

Do we need the "request" packages @rg911? Should fetch be enough? What happens if we remove request?
cannot remember why we used it in the first place. but you are probably right here. Let me have a look

@fboucquez
Copy link
Contributor

Probably leftover from the last request implementation we replaced with fetch. Maybe double-check if there is any other not used dep we can remove?

@rg911 rg911 merged commit a0255f0 into symbol:main Oct 8, 2020
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