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

Merge develop to main branch (v2.4.2) #220

Merged
merged 30 commits into from
Sep 3, 2024
Merged

Merge develop to main branch (v2.4.2) #220

merged 30 commits into from
Sep 3, 2024

Conversation

ShookLyngs
Copy link
Collaborator

What's Changed

Full Changelog: release/v2.4.1...release/v2.4.2

ShookLyngs and others added 30 commits August 19, 2024 13:26
@ShookLyngs
fix: typo in name of TransactionProcessor.getPendingOuputCellsByTxid
1c2f169
@ShookLyngs
refactor: separate the xudt balance calculation of total/pending amount
ed47880
@ShookLyngs
chore: add description and notes for the bitcoin.getAddressTxs() limi…
776ec6d 
…tations
@Flouse
fix: amount calculation in the rgbpp address balance endpoint (#206)
50ff6c4 
Merge pull request #206 from ckb-cell/fix/202-unconfirmed-xudt-balance
@ShookLyngs
fix: validate L1-L1 and L1-L2 isomorphic transactions in depth for th…
7d7a32f 
…e activity API
@ShookLyngs
refactor: rename "ckbRawTx" to "ckbVirtualTx" in the IsomorphicTransa…
acdaa58 
…ction type
@ShookLyngs
refactor: improve code readability in activity-related routes and fun…
a13e5b3 
…ctions
@ShookLyngs
fix: btcInput position does not matter when comparing with ckbTx in i…
808b383 
…sIsomorphicTx()
@ShookLyngs
fix: ckbTx.outputs is allowed to include no type-cells in isIsomorphi…
e0c1c51 
…cTx()
@Flouse
Fix duplicated bindings in the activity API (#194)
9ba1b70 
Merge pull request #194 from ckb-cell/fix/182-duplicated-bindings
@ShookLyngs
refactor: optimize performance of RgbppCollector.queryBtcTimeLockTxBy…
a7b3553 
…BtcTxId()
@ShookLyngs
chore: update activity api snapshot
b1277d4
@ShookLyngs
refactor: rename queryBtcTimeLockTxByBtcTxId to queryBtcTimeLockTxByB…
71d3d59 
…tcTx in RgbppCollector
@ShookLyngs
Merge pull request #212 from ckb-cell/ref/btc-time-lock-cells-searching
6ad5695 
refactor: optimize performance of `queryBtcTimeLockTxByBtcTx()`
@ShookLyngs
refactor: use IS_MAINNET/TESTNET_TYPE constants and related utils in …
cb25b92 
…code, and fix minor typos (#211)
@ShookLyngs
fix: route params should have empty/length validation
7712bc3
@ShookLyngs
refactor: use Zod.safeParse() and throw error via httpErrors.badReque…
3374b6a 
…st() in the validatorCompiler()
@ShookLyngs
fix: the return structure in validatorCompiler() is wrong
987a3e0
@ShookLyngs
fix: prompt a default param name when path is empty in validatorCompi…
70a2d15 
…ler()
@ShookLyngs
refactor: throw badRequest when post rgbpp tx api params is invalid
ad85d9c
@ShookLyngs
chore: update snapshot
57c304e
@Flouse
fix: missing basic validation in route params (#213)
38224cc 
Merge pull request #213 from ckb-cell/fix/210-empty-param-validation
@ShookLyngs
fix: incorrect count of delayed jobs in the TransactionProcessor.enqu…
0e1969b 
…eueTransaction test
@ShookLyngs
chore: upgrade dependencies in the test ci
fd4d2be
@ahonn
Merge pull request #217 from ckb-cell/fix/208-jobs-delayed-count
32389cb 
fix: incorrect count of delayed jobs in the enqueueTransaction test
@ShookLyngs
chore: bump devbox dependencies and pnpm-lock file version
44fa5ca
@ShookLyngs
chore: bump axios from 1.6.7 to 1.7.7
fe7f2e5
@ShookLyngs
Merge pull request #219 from ckb-cell/chore/update-devbox
ce24dca 
chore: bump devbox dependencies, also bump axios to 1.7.7
@ShookLyngs
chore: bump version to v2.4.2
f97a989
@ShookLyngs
Merge pull request #216 from ckb-cell/release/v2.4.2
039a350 
chore: bump version to v2.4.2
@vercel Vercel
Copy link

vercel bot commented Sep 3, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
btc-assets-api ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 3, 2024 5:02am

ahonn
ahonn approved these changes Sep 3, 2024
View reviewed changes
Copy link
Collaborator

@ahonn ahonn left a comment

Choose a reason for hiding this comment

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

LGTM

@Flouse Flouse merged commit 91e9184 into main Sep 3, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ahonn ahonn ahonn approved these changes

@Flouse Flouse Flouse approved these changes

@Dawn-githup Dawn-githup Awaiting requested review from Dawn-githup

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ShookLyngs @Flouse @ahonn