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

Add support for _origin implicit transition parameter #921

Merged
merged 3 commits into from
Dec 16, 2020
Merged

Conversation

vaivaswatha
Copy link
Contributor

Similar to _sender, all transitions now have an implicit paramter _origin, to denote the source of a transaction. The blockchain now needs to supply this value similar to how it supplies _sender. JSON interactions (such as in the IDE) will need to to be updated too.

Copy link
Contributor

@anton-trunov anton-trunov left a comment

Choose a reason for hiding this comment

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

I may have missed it (it's a lot of formatting changes) but it looks like we don't have a test case where _sender and _origin are different e.g. because of a chain call.

@vaivaswatha
Copy link
Contributor Author

I may have missed it (it's a lot of formatting changes) but it looks like we don't have a test case where _sender and _origin are different e.g. because of a chain call.

You're right. I'm now adding a test for this.

Copy link
Contributor

@renlulu renlulu 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 so helpful, thanks!

vaivaswatha added a commit to Zilliqa/scilla-docs that referenced this pull request Dec 16, 2020
vaivaswatha added a commit to Zilliqa/scilla-docs that referenced this pull request Dec 16, 2020
* Document _origin.

Reference: Zilliqa/scilla#921

* Apply suggestions from code review

Co-authored-by: jjcnn <38888011+jjcnn@users.noreply.github.com>

Co-authored-by: jjcnn <38888011+jjcnn@users.noreply.github.com>
@vaivaswatha vaivaswatha merged commit 549c9e6 into master Dec 16, 2020
@vaivaswatha vaivaswatha deleted the txn_origin branch December 16, 2020 07:52
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.

5 participants