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

Upgrade to LiquiDEX v1 #6

Merged
merged 1 commit into from
Oct 18, 2022
Merged

Upgrade to LiquiDEX v1 #6

merged 1 commit into from
Oct 18, 2022

Conversation

LeoComandini
Copy link
Owner

@LeoComandini LeoComandini commented Aug 24, 2022

This is a breaking change, removing support for LiquiDEX v0 and
adding support for LiquiDEX v1.

  • update the proposal format: replace value blinding factors with
    blind value proofs and add scalar offsets.
  • remove liquidex_unblind
  • remove liquidex_assets* functions
  • remove RawStore

Code will be improved once ElementsProject/rust-elements#146 and BlockstreamResearch/rust-secp256k1-zkp#55 will be available from upstream.

src/liquidex.rs Outdated Show resolved Hide resolved
@LeoComandini LeoComandini force-pushed the liquidexv1 branch 5 times, most recently from 73c9d58 to cad529b Compare September 8, 2022 16:29
@LeoComandini LeoComandini changed the title WIP: Upgrade to LiquiDEX v1 Upgrade to LiquiDEX v1 Sep 8, 2022
@LeoComandini LeoComandini force-pushed the liquidexv1 branch 2 times, most recently from 5b76e27 to 3c8ad18 Compare September 14, 2022 08:11
@RCasatta
Copy link

utACK 3ab2a17 , light review

@valerio-vaccaro
Copy link

utack 3ab2a17

This is a breaking change, removing support for LiquiDEX v0 and
adding support for LiquiDEX v1.

* update the proposal format: replace value blinding factors with
  blind value proofs and add scalar offsets.
* remove `liquidex_unblind`
* remove `liquidex_assets*` functions
* remove `RawStore`
@LeoComandini
Copy link
Owner Author

squashed

@LeoComandini LeoComandini merged commit f5c5b06 into master Oct 18, 2022
@LeoComandini LeoComandini deleted the liquidexv1 branch October 18, 2022 10:24
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.

None yet

3 participants