Skip to content
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

Bug: correctly handle denoms with / in pcli tx lp #4766

Merged
merged 1 commit into from
Jul 25, 2024
Merged

Commits on Jul 25, 2024

  1. Bug: correctly handle denoms with / in pcli tx lp

    The parsing looked for the first occurrence of the slash for
    
    nX@nY/fee
    
    but if X or Y contains a slash, this will be incorrect.
    
    This fixes this somewhat, by looking for the last slash, and only
    considering that a fee if it ends with bps.
    
    This will not correctly handle a denom containing both a slash and
    ending with bps, but beyond that we'll need to change the format
    probably.
    cronokirby committed Jul 25, 2024
    Configuration menu
    Copy the full SHA
    4536375 View commit details
    Browse the repository at this point in the history