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

chore: add xhr2 package #2

Closed
wants to merge 10 commits into from

Conversation

im-adithya
Copy link

This should fix the error described in haveno-dex/haveno#1482.

To use:
npm install
npm pack

and install the packaged file in sample app
npm install haveno-ts-0.0.30.tgz
npm run start

You will not see the XMLHttpRequest is not defined error anymore

@im-adithya im-adithya requested a review from woodser as a code owner December 19, 2024 10:49
@im-adithya
Copy link
Author

@woodser please review :)

Copy link

@bl4ck-cod3 bl4ck-cod3 left a comment

Choose a reason for hiding this comment

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

All this does is override the XMLHttpRequest object that is being used with a global object. this introduces multiple other issues and does not solve the initial problem of XMLHttpRequest being called in specific places and being undefined. Furthermore global variables create a maintenance nightmare.

@im-adithya
Copy link
Author

@woodser I thought it would be good to bundle xhr2 along so it comes with haveno-ts, let me know if you have any concerns about XMLHttpRequest being a global object

@woodser
Copy link
Owner

woodser commented Dec 27, 2024

Moved to haveno-dex#339

@woodser woodser closed this Dec 27, 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.

3 participants