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

refactor update stackstransaction constructor to options object #1751

Conversation

janniks
Copy link
Collaborator

@janniks janniks commented Oct 23, 2024

This PR was published to npm with the version 6.14.1-pr.87+6d66e7a4
e.g. npm install @stacks/[email protected]+6d66e7a4 --save-exact

  • Rename StacksTransaction, so we can use the name for a better more human readable alternative in the future
  • Update to accept an options object instead

Copy link

vercel bot commented Oct 23, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
stacksjs-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 25, 2024 7:05pm

Copy link

codecov bot commented Oct 23, 2024

@@ -40,3 +40,5 @@ export type NonFungiblePostCondition = {
};

export type PostCondition = StxPostCondition | FungiblePostCondition | NonFungiblePostCondition;

export type PostConditionModeName = 'allow' | 'deny';
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

@janniks janniks force-pushed the 10-23-refactor_remove_unneeded_deserialize_overloads branch from 4db76a6 to 8a72ccb Compare October 24, 2024 15:12
@janniks janniks force-pushed the 10-23-refactor_update_stackstransaction_constructor_to_options_object branch from 7144c1b to 5ed8a66 Compare October 24, 2024 15:12
@janniks janniks force-pushed the 10-23-refactor_remove_unneeded_deserialize_overloads branch from 8a72ccb to 695a0c7 Compare October 25, 2024 12:05
@janniks janniks force-pushed the 10-23-refactor_update_stackstransaction_constructor_to_options_object branch from 5ed8a66 to 6d66e7a Compare October 25, 2024 12:05
@janniks janniks force-pushed the 10-23-refactor_remove_unneeded_deserialize_overloads branch from 695a0c7 to 8c4466d Compare October 25, 2024 18:55
@janniks janniks force-pushed the 10-23-refactor_update_stackstransaction_constructor_to_options_object branch 2 times, most recently from 44a094e to de8cfc7 Compare October 25, 2024 18:55
@janniks janniks force-pushed the 10-23-refactor_remove_unneeded_deserialize_overloads branch from 8c4466d to 74cb952 Compare October 25, 2024 18:55
@janniks janniks force-pushed the 10-23-refactor_update_stackstransaction_constructor_to_options_object branch from de8cfc7 to c20b7bc Compare October 25, 2024 18:55
@janniks janniks force-pushed the 10-23-refactor_remove_unneeded_deserialize_overloads branch from 74cb952 to 06980cf Compare October 25, 2024 18:55
Base automatically changed from 10-23-refactor_remove_unneeded_deserialize_overloads to next October 25, 2024 18:56
@janniks janniks force-pushed the 10-23-refactor_update_stackstransaction_constructor_to_options_object branch from c20b7bc to 9e16d64 Compare October 25, 2024 18:56
@janniks janniks merged commit 2d9bca3 into next Oct 25, 2024
1 of 3 checks passed
@janniks janniks deleted the 10-23-refactor_update_stackstransaction_constructor_to_options_object branch October 25, 2024 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants