-
Notifications
You must be signed in to change notification settings - Fork 39
Closed
Labels
DevExChange to improve the Developer ExperienceChange to improve the Developer ExperienceenhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomerslowLow priority issueLow priority issue
Description
Few things here:
-
In the
Enter the gas limitquestion:- It is not "gas", it's the
ref_timecomponent of the weight, i.e. the maximum number of picoseconds of execution for a given extrinsic. The name is misleading, and does not have a 1:1 parallelism with the typical Ethereum "gas" devs are used to. I propose renaming if it to something to "execution time" or something alike. Of course, open to discussion. - I wonder... how often users do NOT estimate the call? If users do always estimate, I think it'd make sense to only display this question if the user provides a flag for it, or even in the most extreme case: not specifying it for simplicity.
- It is not "gas", it's the
-
In the
Enter the proof size limitquestion:- If the previous question was left blank... will users really fill this one? I think it should be skipped in that case.
- Same point applies here: it should only be specified if a flag is passed and automatically estimated.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
DevExChange to improve the Developer ExperienceChange to improve the Developer ExperienceenhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomerslowLow priority issueLow priority issue