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

Add Support for P2PKH and P2SH-P2WPKH Address Types #58

Closed

Conversation

notTanveer
Copy link
Contributor

This PR adds support for testing P2PKH and P2SH-P2WPKH address types in the indexer, ensuring silent blocks are correctly processed and retrieved for these transaction types. The test suite has been refactored improving maintainability and extending compatibility with diverse Bitcoin address types.

@notTanveer notTanveer marked this pull request as ready for review November 15, 2024 10:07
e2e/indexer.e2e-spec.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@aruokhai aruokhai left a comment

Choose a reason for hiding this comment

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

Looks good, apart from some minor changes required.

e2e/indexer.e2e-spec.ts Outdated Show resolved Hide resolved
Copy link
Collaborator

@theanmolsharma theanmolsharma left a comment

Choose a reason for hiding this comment

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

Apart from removing hardcoded enum values for types, I don't have any other comments as of now.

e2e/helpers/wallet.helper.ts Outdated Show resolved Hide resolved
@notTanveer notTanveer force-pushed the feature/address-types branch 8 times, most recently from 6c16999 to 8d6bf7c Compare November 25, 2024 15:01
Draft: Improved E2E Test

implemented improved e2e test structure
@notTanveer notTanveer force-pushed the feature/address-types branch 2 times, most recently from c14897a to 5583fc1 Compare November 27, 2024 11:15
e2e/helpers/wallet.helper.ts Outdated Show resolved Hide resolved
e2e/helpers/wallet.helper.ts Outdated Show resolved Hide resolved
e2e/indexer.e2e-spec.ts Outdated Show resolved Hide resolved
@notTanveer notTanveer force-pushed the feature/address-types branch from 70f9a62 to ce30905 Compare November 27, 2024 14:20
@notTanveer notTanveer closed this Dec 13, 2024
@theanmolsharma
Copy link
Collaborator

Why did you close this?

@notTanveer
Copy link
Contributor Author

Why did you close this?

merge conflicts

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