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

fix: blocksyncer adapt event param to chain side and payment module added #205

Merged
merged 3 commits into from
Mar 16, 2023

Conversation

krish-nr
Copy link
Contributor

Description

fix some event properties to keep consistent with chain side.
Repull previous PR to add payment module

Rationale

adapt chain side changes and add a new business module

Example

N/A

Changes

Notable changes:

  • fix some event properties to keep consistent with chain side.
  • add payment module to persist payment related data
  • get config from env as a higher priority way

constwz and others added 2 commits March 15, 2023 14:24
* feat:add block syncer module

* feat: add some small optimize

* feat: use interface method instead of direct call

---------

Co-authored-by: krish-nr <krish.z@nodereal.io>
@krish-nr krish-nr changed the title fix: blocksyncer adapt event param to chain side and payment module added #204 fix: blocksyncer adapt event param to chain side and payment module added Mar 15, 2023
@krish-nr
Copy link
Contributor Author

As previous PR has a wrong target branch(main), this is a repull request to develop branch, changes have been reviewed except the merge with develop branch

@krish-nr krish-nr added the r4r Ready for review label Mar 15, 2023
@krish-nr krish-nr requested review from constwz and joeylichang March 15, 2023 14:22
@joeylichang
Copy link
Contributor

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
r4r Ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants