Skip to content

feat: nbtc minting#38

Merged
sczembor merged 11 commits intomasterfrom
stan/nbtc-mint
Jul 17, 2025
Merged

feat: nbtc minting#38
sczembor merged 11 commits intomasterfrom
stan/nbtc-mint

Conversation

@sczembor
Copy link
Contributor

Description

Closes: #20


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

  • included the correct type prefix in the PR title
  • added ! to the type prefix if API or client breaking change
  • added appropriate labels to the PR
  • provided a link to the relevant issue or specification
  • added a changelog entry to CHANGELOG.md
  • included doc comments for public functions
  • updated the relevant documentation or specification
  • reviewed "Files changed" and left comments if necessary

sczembor added 2 commits July 15, 2025 16:57
Signed-off-by: sczembor <stanislaw.czembor@gmail.com>
Signed-off-by: sczembor <stanislaw.czembor@gmail.com>
@sczembor sczembor requested a review from a team as a code owner July 15, 2025 15:02
sczembor added 2 commits July 15, 2025 17:18
Signed-off-by: sczembor <stanislaw.czembor@gmail.com>
Signed-off-by: sczembor <stanislaw.czembor@gmail.com>
Copy link
Contributor

@robert-zaremba robert-zaremba left a comment

Choose a reason for hiding this comment

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

let's move the Sui code to a separate class to allow better testability (see comments)

sczembor added 3 commits July 16, 2025 12:15
…to a separate modules

Signed-off-by: sczembor <stanislaw.czembor@gmail.com>
Signed-off-by: sczembor <stanislaw.czembor@gmail.com>
Signed-off-by: sczembor <stanislaw.czembor@gmail.com>
Copy link
Contributor

@robert-zaremba robert-zaremba left a comment

Choose a reason for hiding this comment

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

pre-approved

sczembor and others added 4 commits July 16, 2025 14:43
Co-authored-by: Robert Zaremba <robert@zaremba.ch>
Signed-off-by: sczembor <43810037+sczembor@users.noreply.github.com>
Signed-off-by: sczembor <stanislaw.czembor@gmail.com>
Signed-off-by: sczembor <stanislaw.czembor@gmail.com>
Signed-off-by: sczembor <stanislaw.czembor@gmail.com>
@sczembor sczembor merged commit 17663a8 into master Jul 17, 2025
10 checks passed
@sczembor sczembor deleted the stan/nbtc-mint branch July 17, 2025 09:20
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.

nbtc indexing: minting - sui integration and triger nbtc_mint

2 participants