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.

Sign up for GitHub

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

Jump to bottom

Add an example for using a Plutus script as stake credential #3162

Merged
merged 4 commits into from
Oct 7, 2021
Merged

Add an example for using a Plutus script as stake credential #3162

merged 4 commits into from
Oct 7, 2021

Conversation

kantp
Copy link
Contributor

@kantp kantp commented Sep 3, 2021

No description provided.

@Jimbo4350 Jimbo4350 force-pushed the kantp/stake-script-credential-test branch from f2ad65c to 1433fb3 Compare September 3, 2021 18:55
@Jimbo4350
Copy link
Contributor

Jimbo4350 commented Sep 3, 2021 •
edited
Loading

We should not merge the commit in master that enables building of staking script addresses in the cli

@Jimbo4350 Jimbo4350 self-requested a review September 3, 2021 19:06
Jimbo4350
Jimbo4350 approved these changes Sep 3, 2021
View reviewed changes
Copy link
Contributor

@Jimbo4350 Jimbo4350 left a comment

Choose a reason for hiding this comment

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

LGTM! We just need to adapt the scripts to use the local Alonzo cluster. I can take this over if you like.

@kantp
Copy link
Contributor Author

kantp commented Sep 6, 2021

That would be great @Jimbo4350, thanks!

@Jimbo4350 Jimbo4350 force-pushed the kantp/stake-script-credential-test branch 5 times, most recently from e706132 to 6376054 Compare September 21, 2021 19:26
@Jimbo4350
Copy link
Contributor

bors r+

iohk-bors bot added a commit that referenced this pull request Sep 21, 2021
@iohk-bors @Jimbo4350
Merge #3162
6120ecf 
3162: Add an example for using a Plutus script as stake credential r=Jimbo4350 a=kantp



Co-authored-by: Philipp Kant <philipp.kant@iohk.io>
Co-authored-by: Jordan Millar <jordan.millar@iohk.io>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Sep 21, 2021

Build failed:

@mkoura mkoura mentioned this pull request Sep 30, 2021
Closed
54 tasks
@Jimbo4350 Jimbo4350 force-pushed the kantp/stake-script-credential-test branch 2 times, most recently from ea31fac to a0bc10f Compare October 6, 2021 16:54
@Jimbo4350 Jimbo4350 force-pushed the kantp/stake-script-credential-test branch from a0bc10f to 179ec4f Compare October 6, 2021 21:13
@Jimbo4350
Copy link
Contributor

bors r+

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Oct 7, 2021

Build succeeded:

@iohk-bors iohk-bors bot merged commit f60b392 into master Oct 7, 2021
@iohk-bors iohk-bors bot deleted the kantp/stake-script-credential-test branch October 7, 2021 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Jimbo4350 Jimbo4350 Jimbo4350 approved these changes

@DavidEichmann DavidEichmann Awaiting requested review from DavidEichmann

@dcoutts dcoutts Awaiting requested review from dcoutts

@erikd erikd Awaiting requested review from erikd

@JaredCorduan JaredCorduan Awaiting requested review from JaredCorduan

@nc6 nc6 Awaiting requested review from nc6

@newhoggy newhoggy Awaiting requested review from newhoggy

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kantp @Jimbo4350