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

feat(frontend): BTC transactions [GIX-2795] #2547

Merged
merged 12 commits into from
Oct 1, 2024

Conversation

DenysKarmazynDFINITY
Copy link
Contributor

Motivation

This is the base branch for BTC transactions feature.

@DenysKarmazynDFINITY DenysKarmazynDFINITY changed the title feat(frontend): implement service for communicating with Blockchain A… feat(frontend): BTC transactions Sep 26, 2024
@DenysKarmazynDFINITY DenysKarmazynDFINITY changed the title feat(frontend): BTC transactions feat(frontend): BTC transactions [GIX-2795] Sep 27, 2024
@DenysKarmazynDFINITY DenysKarmazynDFINITY force-pushed the feat(frontend)/btc-transactions branch 2 times, most recently from b7ea05d to 7ae3280 Compare September 27, 2024 14:11
@DenysKarmazynDFINITY DenysKarmazynDFINITY marked this pull request as ready for review October 1, 2024 14:28
@DenysKarmazynDFINITY DenysKarmazynDFINITY requested a review from a team as a code owner October 1, 2024 14:28
…PI (#2507)

# Motivation

The goal is to implement service for communicating with Blockchain API.
Initially, it will be used for fetching uncertified BTC transactions.
# Motivation

Remove unnecessary slash from `btcAddressData`.
# Motivation

The goal is to extend BTC networks with already defined explorer URLs.
It will be used in the BTC transaction detail modal.
…-2795] (#2549)

# Motivation

Removing the temporary type for BTC wallet since we are going to re-use
`PostMessageDataResponseWallet`.
# Motivation

BTC transaction modal stores.
# Motivation

Extend `blockchain.rest` with the `btcLatestBlock` function. It will be
used for calculating the number of confirmations for a BTC transaction.
# Motivation

Add BTC transaction stores. It will be later extended with additional
methods, similar to other related stores (e.g. `cleanUp`).
# Motivation

A parser for BTC transactions. It will later extended with additional
field: "confirmations".
# Motivation

The goal is to update the BTC wallet worker to fetch transactions.
Similar to other store, we only load a BTC address if it hasn't been
already fetched by FE.
Copy link
Member

@peterpeterparker peterpeterparker left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@DenysKarmazynDFINITY DenysKarmazynDFINITY merged commit 4842908 into main Oct 1, 2024
8 checks passed
@DenysKarmazynDFINITY DenysKarmazynDFINITY deleted the feat(frontend)/btc-transactions branch October 1, 2024 15:08
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