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: axios import #58

Closed
wants to merge 3 commits into from
Closed

fix: axios import #58

wants to merge 3 commits into from

Conversation

zixiang2018
Copy link
Contributor

@zixiang2018 zixiang2018 commented May 6, 2024

Context

  • CRA using dnsprove in our repo will face an error TypeError: (0 , _axios.default) is not a function
    Screenshot 2024-05-06 at 4 11 13 PM

  • Turns out that importing axios from 'axios' results in the CRA fetching a file - "/static/media/axios.xxxx.cjs"
    Screenshot 2024-05-06 at 3 43 52 PM

What this PR does

  • To fix this issue by refering to the ../node_module/axios to reference a function instead of a file

@zixiang2018
Copy link
Contributor Author

Refer to #59

@zixiang2018 zixiang2018 closed this May 6, 2024
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