Skip to content
This repository has been archived by the owner on Oct 21, 2024. It is now read-only.

feat(spv-642): return ArcError to prevent infinite broadcasting of invalid transactions #94

Merged
merged 7 commits into from
May 15, 2024

Conversation

arkadiuszos4chain
Copy link
Contributor

@arkadiuszos4chain arkadiuszos4chain commented Apr 19, 2024

Pull Request Checklist

  • πŸ“– I created my PR using provided : CODE_STANDARDS
  • πŸ“– I have read the short Code of Conduct: CODE_OF_CONDUCT
  • 🏠 I tested my changes locally.
  • βœ… I have provided tests for my changes.
  • πŸ“ I have used conventional commits.
  • πŸ“— I have updated any related documentation.
  • πŸ’Ύ PR was issued based on the Github or Jira issue.

@arkadiuszos4chain arkadiuszos4chain self-assigned this Apr 19, 2024
Copy link

github-actions bot commented Apr 19, 2024

Manual Tests

πŸ’š Manual testing by @wregulski resulted in success.

@arkadiuszos4chain arkadiuszos4chain marked this pull request as draft April 19, 2024 06:43
@arkadiuszos4chain arkadiuszos4chain marked this pull request as ready for review April 19, 2024 07:28
Nazarii-4chain
Nazarii-4chain previously approved these changes Apr 19, 2024
chris-4chain
chris-4chain previously approved these changes Apr 19, 2024
@wregulski wregulski self-assigned this May 14, 2024
@wregulski wregulski added the tested PR was tested by a team member label May 15, 2024
@wregulski
Copy link
Contributor

I marked is as tested manually, it was tested by @pawellewandowski98 along with SPV-Wallet

@wregulski wregulski merged commit d4594f5 into main May 15, 2024
3 checks passed
@wregulski wregulski deleted the fix-642-broadcast branch May 15, 2024 14:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
tested PR was tested by a team member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants