Skip to content

Conversation

jowlee
Copy link
Contributor

@jowlee jowlee commented Nov 9, 2021

Description

Update the number of decimals for updated wrapped audio from wormhole v2
Switches from 9 decimals to 8 decimals

Tests

How will this change be monitored?

Closes AUD-1059

@jowlee jowlee added libs discovery-node Discovery Node (previously known as Discovery Provider) labels Nov 9, 2021
@jowlee jowlee requested a review from piazzatron November 9, 2021 16:39
@jowlee jowlee changed the title Jowlee waudio decimals waudio decimals Nov 10, 2021
Copy link
Contributor

@piazzatron piazzatron left a comment

Choose a reason for hiding this comment

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

We're missing one at the top of challenge.js - WRAPPED_AUDIO_PRECISION.

I think it would be great if we could do a pass here and consolidate this to like one single var per repo - once per libs, once for DN, etc. Feels like we're just asking to mess something up otherwise.

@jowlee jowlee force-pushed the jowlee-waudio-decimals branch from f6d32b8 to f07a881 Compare November 29, 2021 15:15
@jowlee jowlee force-pushed the jowlee-waudio-decimals branch from 45221e2 to 6fd332f Compare November 29, 2021 15:58
@jowlee jowlee merged commit 94dca47 into master Nov 29, 2021
@jowlee jowlee deleted the jowlee-waudio-decimals branch November 29, 2021 17:11
@jowlee jowlee mentioned this pull request Dec 1, 2021
@AudiusProject AudiusProject deleted a comment from linear bot Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

discovery-node Discovery Node (previously known as Discovery Provider) libs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants