Skip to content
This repository has been archived by the owner on Dec 16, 2021. It is now read-only.

refactor: broadcastTransaction without insight #346

Merged
merged 9 commits into from
Mar 9, 2021
Merged

refactor: broadcastTransaction without insight #346

merged 9 commits into from
Mar 9, 2021

Conversation

Alex-Werner
Copy link
Contributor

@Alex-Werner Alex-Werner commented Mar 1, 2021 •
edited
Loading

Issue being fixed or feature implemented

This PR brings support for broadcastTransaction without insight

What was done?

  • feat: provide broadcastTransaction using core RPC instead of insight

How Has This Been Tested?

Using test suite

Breaking Changes

N/A

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

@Alex-Werner Alex-Werner added this to the v0.18 milestone Mar 1, 2021
@Alex-Werner Alex-Werner marked this pull request as ready for review March 1, 2021 12:25
@shumkov shumkov changed the base branch from v0.18-dev to v0.19-dev March 3, 2021 07:42
@shumkov shumkov modified the milestones: v0.18, v0.19 Mar 3, 2021
@Alex-Werner Alex-Werner requested a review from shumkov March 9, 2021 04:36
@Alex-Werner Alex-Werner requested a review from shumkov March 9, 2021 05:30
shumkov
shumkov approved these changes Mar 9, 2021
View reviewed changes
Copy link
Member

@shumkov shumkov 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! 👍

@shumkov shumkov merged commit 9760a6f into v0.19-dev Mar 9, 2021
@shumkov shumkov deleted the feat/broadcast-tx branch March 9, 2021 05:52
@shumkov shumkov changed the title feat: provide broadcastTransaction without insight refactor: broadcastTransaction without insight Mar 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@shumkov shumkov shumkov approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
v0.19
Development

Successfully merging this pull request may close these issues.
2 participants
@Alex-Werner @shumkov