-
Notifications
You must be signed in to change notification settings - Fork 142
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
feat: Add GetNewAddress gRPC API #1193
feat: Add GetNewAddress gRPC API #1193
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #1193 +/- ##
==========================================
- Coverage 81.44% 81.42% -0.02%
==========================================
Files 179 179
Lines 9500 9531 +31
==========================================
+ Hits 7737 7761 +24
- Misses 1381 1384 +3
- Partials 382 386 +4 |
fa8d4ab
to
4c9a9e2
Compare
www/grpc/proto/wallet.proto
Outdated
|
||
// Enum for the address type. | ||
enum AddressType { | ||
ADDRESS_TYPE_ADDRESS_TYPE_TREASURY = 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it can be ADDRESS_TYPE_TREASURY
or even TREASURY
. Same for other values
// Response message containing the new address. | ||
message GetNewAddressResponse { | ||
string wallet_name = 1; | ||
AddressInfo address_info = 2; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please comment these lines.
They will be included in the automatically generated documents.
4c9a9e2
to
115aa71
Compare
Description
This PR Add
GetNewAddress
API endpoint