Skip to content

Conversation

@auroter
Copy link
Contributor

@auroter auroter commented Mar 26, 2023

I've added a tracer on the Avalanche Testnet (Fuji). The Quote endpoint is using it to find all addresses associated with a requested transaction.

To test:

  1. Request a quote on the Avalanche Testnet (43113)
  2. The default quote in Postman should print the address of the wallet we are minting to

you will need to restart docker because there is a new migration file and updated data seeding

ocasta181
ocasta181 previously approved these changes Mar 27, 2023
Copy link
Contributor

@ocasta181 ocasta181 left a comment

Choose a reason for hiding this comment

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

ADDRESSES USED IN QUOTE: [0xDf2b71D831476DbCff5f9DCed068A0c6305A6C54]{"level":"info","path":"/heartbeat","method":"GET","status_code":200,"request_id":"QkcUZIGSm7PfTOBUO0SzELEkvUDrjCm7","host":"localhost:5555","latency":0.031917,"env":"local","message":"request"}

@auroter auroter changed the base branch from develop to task/andrin/str-352 March 28, 2023 19:06
@auroter auroter requested a review from ocasta181 March 28, 2023 19:06
if err != nil {
return res, 0, libcommon.StringError(err)
}
fmt.Printf("\n\nADDRESSES USED IN QUOTE: %v", addresses)

Choose a reason for hiding this comment

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

same as above

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is being used to print the output for testing this PR. It will be removed in my next task, of implementing the tracer logic.

ocasta181
ocasta181 previously approved these changes Mar 28, 2023
@ocasta181 ocasta181 self-requested a review March 28, 2023 19:43
@saito-sv saito-sv self-requested a review March 28, 2023 20:00
@auroter auroter merged commit 58df852 into task/andrin/str-352 Mar 28, 2023
@auroter auroter deleted the task/sean/str-499 branch March 28, 2023 23:35
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.

5 participants