-
Notifications
You must be signed in to change notification settings - Fork 198
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
En 7295 get username route #2203
Conversation
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.
You started from the wrong branch. Your changes include relayed-tx-bugfix as well.
} | ||
|
||
addr, err := n.addressPubkeyConverter.Decode(address) | ||
account, err := n.getAccountHandler(address) |
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.
GJ! Same level of abstraction
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.
System tests passed
Added an API endpoint for getting the username for a given address.
Testing steps:
a) Prerequisites
-a testnet which has the following routes enabled in
api.toml
:-
address/:address/username
-
address/:address
b) Test steps
-make API requests to the routes
c) Expected results
-on the
address/:address/username
route only the username should be returned-the username should be visible also on the route
address/:address
-if the given address exists and it has an username set, it should be returned.
-if the given address does not exist an error will be returned.
-if the given address exists, but it doesn't have an username set, an empty username will be returned.
d) Actual results
-tested on a local testnet and the acceptance criteria is met