-
Notifications
You must be signed in to change notification settings - Fork 276
-
Notifications
You must be signed in to change notification settings - Fork 276
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
[suggestion] Move NewParameter
into Executor
#3901
Comments
I think that would be right |
Signed-off-by: Dmitry Murzin <diralik@yandex.ru>
…or `migrate` entrypoint Signed-off-by: Dmitry Murzin <diralik@yandex.ru>
What means "move As far as I understand, Executor just has access to all parameters, to the key-value storage with some data. And, Executor just "interprets" certain keys in a certain way which defines Executor's behaviour. Is it needed for Executor to define "known" parameters in some way? |
just look at how permission tokens are defined in Executor. We need to replicate that mechanism |
This instruction has been removed, now there is only |
We should consider removing
NewParameter
ISI and only allow defining new parameters in Executor. During Executor migration old parameters will be removed and new ones can be added. The reason is that having an ISI doesn't make sense for an operation that can only be executed by the blockchain adminThe text was updated successfully, but these errors were encountered: