Chenna Keshava B S
Chenna Keshava B S
@ckniffen I have added the algorithm data member in this commit: [a234fa9](https://github.com/XRPLF/xrpl.js/pull/2658/commits/a234fa9cc9a03231f9b84651e805eebcc8365eba) Please review at your convenience
@achowdhry-ripple can you please review this PR at your convinience?
@mvadari Yes, I agree with you. Regarding removing the `definitions.json` file entirely: we will need to perform an RPC call once to fetch the `server_definitions` output and then use it...
FYI: Node 18 has LTS support only until April 2025. We will need to perform a similar obviation of Node 18 at that time
#2678 has been merged
@Bronek thanks for the quick review! What do you mean by USER context? I feel the `getLedgerByContext` should only be used by the consensus mechanism (and Full History Nodes) to...
It is the consensus of the xrpl.js developers that this is not an issue. Javascript developers are expected to understand the numeric limitations of the `number` class and use it...
Here is a sample package that's published by this Github Actions workflow: https://pypi.org/project/xrpl-py-release-test/#description Let me know if something is amiss. I verified that this package is published correctly, through `pip...
I need to use some tags to test the publishing of "tagged commits". I will delete these test-tags once my testing is complete.
For some reason, the CodeRabbit AI reminds me of the bunny in Zootopia :)), but helpful suggestions. Thanks.