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

Updated Week1 code (EnglishAuction) to newer version of Plutus #21

Closed
wants to merge 2 commits into from

Conversation

nemo83
Copy link

@nemo83 nemo83 commented May 26, 2021

Updated EnglishAuction code to use more recent version of Plutus

As I am planning to implement an Auction Smart Contract myself, I've decided to give a shot and try to update the EnglishAuction code to better understand what's going on. Hopefully this could turn out handy for the next wave of Plutus Pioneer.

Of course I don't mind if the PR is rejected, but a review would be more than appreciated to check whether or not there are errors or improvements.

TODO:

  • Add more EmulatorTrace tests
  • Fix the issue whereby the code doesn't compile anymore in the playground (will provide the error)

Currently the error I can't fix is:
Screenshot 2021-05-26 at 11 32 01

@nemo83 nemo83 changed the title Updated code to newer version of Plutus [DRAFT] Updated code to newer version of Plutus May 26, 2021
@nemo83 nemo83 changed the title [DRAFT] Updated code to newer version of Plutus [DRAFT] Updated Week1 code (EnglishAuction) to newer version of Plutus May 26, 2021
Comment on lines 355 to 356
test :: IO ()
test = runEmulatorTraceIO' def emCfg myTrace
Copy link
Author

@nemo83 nemo83 May 26, 2021

Choose a reason for hiding this comment

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

Planning to add more test combinations.

@nemo83 nemo83 changed the title [DRAFT] Updated Week1 code (EnglishAuction) to newer version of Plutus Updated Week1 code (EnglishAuction) to newer version of Plutus May 26, 2021
@mzfr mzfr mentioned this pull request Nov 4, 2021
@luisantonioig
Copy link
Contributor

Hi @nemo83,

I hope this message finds you well.

I wanted to inform you that we have decided to close this pull request as it pertains to an old branch of the course. Please note that these branches are not actively maintained and are kept for historical purposes.

We encourage you to utilize the branch that best suits your needs, whether it's the current version or any other relevant branch.

Please don't hesitate to reach out if you have any questions or require further assistance. We're here to help!

Thank you for your understanding and your valuable contributions.

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

2 participants