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

Error with search bar #36

Open
sa-egor opened this issue Feb 21, 2019 · 7 comments
Open

Error with search bar #36

sa-egor opened this issue Feb 21, 2019 · 7 comments

Comments

@sa-egor
Copy link

sa-egor commented Feb 21, 2019

Hey guys!

I got same error: #20 in search bar. I trying search address but i see UI trying search block and I see 404 error in console: ".../insight-api/block/MUKtt1tiVmihethhgXVn5PMusyRPwJ61eM 404 (Not Found)".
This example address for my coin: MUKtt1tiVmihethhgXVn5PMusyRPwJ61eM
I use insight-ui for my coin based on Dash.
I snifed traffic from insight-api to my coind, and I see what coind return error with code: 500.
Traffic screenshot:
screenshot_20190221_230303
Can because of that be a problem with the search?

I use:
insight-api v2.0.1
insight-ui 2.0.0

Thanks! Best regards!

@BenjaminNitschke
Copy link

actually all searches will go to /block/ instead of /address/ for addresses and /tx/ for transactions, this is broken in the current release, worked fine before.

we will fix it with our next update to https://mydashwallet.org:3001/insight (and soon https://explorer.mydashwallet.org ) and provide a pullrequest upstream so insight.dash.org can also get fixed. See our repo with many more changes at: https://github.com/deltaengine/insight-ui

@sa-egor
Copy link
Author

sa-egor commented Mar 4, 2019

Thanks for answer. I'll wait this update.

@BenjaminNitschke
Copy link

Seems to be fixed on https://insight.dash.org, not sure how they did it, github code didn't change.

We tried fixing it several ways, but it is still not very good (address and tx sometimes work, block search is a bit broken, seems like no/incomplete errors are not leading to the correct search): https://explorer.mydashwallet.org

Other than that our explorer is much nicer ^^ will issue a pull request later

@sa-egor
Copy link
Author

sa-egor commented Apr 8, 2019

Any news?

@BenjaminNitschke
Copy link

sorry, no good fix (we changed the default to addresses, which is what most users look for). rolling back to a version from about 2 months ago also works: https://explorer.mydashwallet.org

Some of the changes can be seen at https://github.com/deltaengine/insight-ui we will probably do a bit more later this week, but wasted a lot of time today on crashed nodes (mixing and explorer nodes both crashed, probably too many tx) and broken insight explorer crap already.

@bfoss765
Copy link

Added to backlog.

@Alex-Werner
Copy link

Hi guys,

The issue should be fixed in #52 and will find release after review, do not hesitate to give an eye, but that's some basic regex validation.

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

No branches or pull requests

4 participants