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.

Sign up for GitHub

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

Jump to bottom

feat(SPV-848) notifications #246

Merged
merged 18 commits into from
Jul 12, 2024
Merged

feat(SPV-848) notifications #246

merged 18 commits into from
Jul 12, 2024

Conversation

chris-4chain
Copy link
Contributor

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.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 28, 2024 •
edited
Loading

Manual Tests

💚 Manual testing by @pawellewandowski98 resulted in success.

@chris-4chain chris-4chain marked this pull request as ready for review July 2, 2024 10:06
@chris-4chain chris-4chain requested a review from a team as a code owner July 2, 2024 10:06
@chris-4chain chris-4chain mentioned this pull request Jul 2, 2024
Merged
7 tasks
dorzepowski
dorzepowski requested changes Jul 8, 2024
View reviewed changes
examples/go.mod Outdated Show resolved Hide resolved
go.mod Outdated Show resolved Hide resolved
examples/list_transactions/list_transactions.go Outdated Show resolved Hide resolved
notifications/registerer.go Outdated Show resolved Hide resolved
notifications/registerer.go Outdated Show resolved Hide resolved
examples/webhooks/webhooks.go Show resolved Hide resolved
@pawellewandowski98 pawellewandowski98 added the tested PR was tested by a team member label Jul 12, 2024
@chris-4chain chris-4chain added the feature Any new significant addition label Jul 12, 2024
@chris-4chain chris-4chain merged commit 44c5921 into main Jul 12, 2024
9 checks passed
@chris-4chain chris-4chain deleted the feat-848-webhook branch July 12, 2024 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dorzepowski dorzepowski dorzepowski approved these changes

@wregulski wregulski wregulski approved these changes

Assignees
No one assigned
Labels
feature Any new significant addition tested PR was tested by a team member
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@chris-4chain @dorzepowski @wregulski @pawellewandowski98