Skip to content

Allowed triggers in attributes#763

Closed
shargon wants to merge 2 commits intoneo-project:masterfrom
shargon:centralize-trigger
Closed

Allowed triggers in attributes#763
shargon wants to merge 2 commits intoneo-project:masterfrom
shargon:centralize-trigger

Conversation

@shargon
Copy link
Copy Markdown
Member

@shargon shargon commented May 20, 2019

Is just an idea, if you want we can centralize the syscall rights with attributes.
Btw, PutEx is only allowed on Application, System is not the same?

@shargon shargon requested a review from erikzhang May 20, 2019 08:23
@erikzhang
Copy link
Copy Markdown
Member

This seems a bit cumbersome. Let's keep it as it is.

@shargon
Copy link
Copy Markdown
Member Author

shargon commented May 20, 2019

ok, but PutEx is not expected in System ?

@erikzhang
Copy link
Copy Markdown
Member

System trigger is for native contracts only. And native contracts never call SYSCALL directly.

@shargon shargon closed this May 20, 2019
@shargon shargon deleted the centralize-trigger branch May 20, 2019 09:01
Thacryba pushed a commit to simplitech/neo that referenced this pull request Feb 17, 2020
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.

2 participants