-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat: upgrade axios #18
Conversation
69f9f9e
to
d1ed7cf
Compare
(updated commit message) |
Could you try to test these changes by importing them into a new application and verify that the functions calling For more context, the creator website is not working with these changes. |
@rongquan1 (added in protocol check in case https:// or http:// is used on dnsquery) is there something specific that broke ? |
Could you please take a look at the We are encountering the following error: |
is it on the editor side or during the build process ? seems to work when importing it locally
build and lint works as well |
This is happening at runtime when we try to click a button on the UI which is supposed to call the function that triggers the axios. |
Fixed, can try again |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Can you update the tt-verify
repo after this package is published?
sure! |
the release seems to be failing, seems to be an npm token issue |
🎉 This PR is included in version 2.13.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
@osslgtm package just released |
Using relative path |
Do you have details on the issue happening ? Would you prefer if we patch the issue or use an alternate package ? Any info would be appreciated |
We are open to using an alternative package to replace However, we must ensure that the replacement maintains all existing functionality. |
upgrade axios