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

Bisq support #92

Merged
merged 33 commits into from Jul 19, 2020
Merged

Bisq support #92

merged 33 commits into from Jul 19, 2020

Conversation

softsimon
Copy link
Member

@softsimon softsimon commented Jul 14, 2020

This PR adds a new optional module that supports exploring the Bisq DAO as per bisq-network/projects#37

softsimon and others added 26 commits July 3, 2020 23:45
Transaction view.
Block view.
Blocks list.
Address view.
Allow search for block height.
Hide empty bisq blocks.
BSQ prices.
API for Bisq block height tip.
Loading indicators on transactions/blocks view.
Total sent now correctly display burnt on Pay trade fee txs.
Adding divider to network dropdown.
Watching partent bisq dump folder.
@Emzy
Copy link
Member

Emzy commented Jul 18, 2020

Tested ACK
I have installed this pr on htps://mempool.emzy.de looks good to me.
Compared some BSQ transactions, all look the same.
Did no code review.

@wiz
Copy link
Member

wiz commented Jul 18, 2020

@softsimon Thanks for your hard work, the new Bisq explorer is amazing!

AFAIK the only remaining show stopper issues are:

  • Fix pagination issue preventing scroll back to last page, i.e. /bisq, /bisq/blocks, etc.
  • Fix API docs on About page, i.e. /api/v1/bisq -> /bisq/api
  • Fix mobile layout so /bisq fits and it does not scroll horizontally
  • Add missing Layer 1 data to Bisq TX page, i.e. mining fees, etc.

Copy link
Member

@wiz wiz left a comment

Choose a reason for hiding this comment

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

ACK

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.

None yet

3 participants