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

problems with storing emissions on goerli from fabric #75

Closed

sichen1234 opened this issue Feb 4, 2021 · 1 comment

Assignees

Contributor

sichen1234 commented Feb 4, 2021

Trying the http://localhost:9000/api-docs/#/emissionscontract/recordAuditedEmissionsToken to store emissions on goerli from fabric, I noticed a few issues:

Why is this API GET and not PUT?
It should have utilityId and partyId to store only the emissions of one party
The API should display emissionsAmount not energyUseAmount here in the log file:
The quantity of tokens issued is wrong: The emissions amount in from the response is 0.07397955129208716 but on Goerli it shows a quantity of 7000

sichen1234 assigned jamesondh

Contributor Author

sichen1234 commented Feb 11, 2021

A couple of changes after today's testing:

store emissions records with UUID as ID instead of taking ID as a paramter of recordEmissions
change recordAuditedEmissionsToken to accept a list of these UUID's (or a string separated by ",") and then find those emissions records and tokenize them, and remove all the party id, utility id, from date, thru date in the lookup.

sichen1234 closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment