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

Doc: add a first page on double-satisfaction #4280

Merged
merged 1 commit into from Dec 15, 2021

Conversation

michaelpj
Copy link
Contributor

I'm going to bulk this section up to include more pages on other kinds
of weakness in the next few days.

I'm going to bulk this section up to include more pages on other kinds
of weakness in the next few days.
@thealmarty thealmarty self-requested a review December 13, 2021 17:00
Copy link
Contributor

@thealmarty thealmarty left a comment

Choose a reason for hiding this comment

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

This is good to merge as is. Other weaknesses can be added in other PRs.


This output can only be spent if, in the same transaction, there is an output sending the agreed-upon payment (encoded in the output’s datum) to A.

Now suppose that A and B have two swaps in progress, one for a token T1 at the price of 10 Ada, and one for a token T2 at the same price.
Copy link

@yihuang yihuang Dec 14, 2021

Choose a reason for hiding this comment

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

Do you mean A creates two outputs that sell token T1 at the price of 10 Ada, and sell token T2 at the same price?

@michaelpj michaelpj merged commit 0f8334a into master Dec 15, 2021
@kwxm kwxm deleted the mpj/double-satisfaction-doc branch February 2, 2022 08:44
MaximilianAlgehed pushed a commit to Quviq/plutus that referenced this pull request Mar 3, 2022
I'm going to bulk this section up to include more pages on other kinds
of weakness in the next few days.
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