-
Notifications
You must be signed in to change notification settings - Fork 157
improve submit transaction #1155
Comments
Another one for you Charlotte. I think Andy Hurt or Liam Grace would be good folks to explain to you what's needed here :) Similar plan to the events one, do some research, come up with some suggestions, and then let's review both of them in one meeting with the VS Code squad 👍 |
Would be great if my previous inputs were selectable below (in the style of the selecting a function) and also if the form could tell me the arg names not just to enter arg0 etc |
Transaction web view update This is the latest click through of the transactions: I've since had further feedback from Liam which I'll be acting on today/tomorrow. I have also been working with @erin-hughes to find out what i need to be doing to make creating this web view easier for her. |
Great progress on this @Charlotte8699 . I wonder how it would look without the twisties in the "Recent" view? I think a permanently displayed table could work here, with new transactions appearing at the top - collapsing by date doesn't seem to add much to me! |
Has any thought been put into how i might run this using keyboard shortcuts? Ideally I would like to look at my code whilst I run the command. I usually do most of my repeatable transactions when i am in debug mode and like to stay on the same code page so I can view the values in variables as it goes through breakpoints. The current method of runnign transactions is nice as I can do it entirely from my keyboard. CMD+L I have configured to bring up the Evaluate vs Submit options and can then use arrow keys and enter to select the transaction and run it. |
Latest development - deleting the date filter, showing how the filters would work and switching between the alternatives for entering in arguments. https://xd.adobe.com/view/52870c3d-9c4c-47ef-7312-798420c1556b-24b6/ @awjh I have thought briefly how shortcuts could be included but as of yet, i haven't put too much thought into it. I'll come find you, Caroline, Jake etc. once we've settled on the layout to plan how commands could be used. I'll put time into this next week! |
I don't think you would want the alternatives for arguments as you don't have key value pairs for arguments, this would work for transient data though. |
With the work happening in #1510, we'll also need to be able to handle peer-targeting in the new transaction page. |
Changed the format for the custom peers and adjusted the wording on the filters. Hoping to get a flow for this done today. |
Created a flow for when a user would create a transaction, save a transaction and then use the filters. If there are other flows people would like to see, let me know. @sstone1 Ed recommended I show you this. |
@Charlotte8699 i noticed that export was only on a saved one, just wondered if users might also want to export on none saved ones? I can see an argument for both sides though. Can we get a flow for import, i remember us talking about that i wasn't sure how it would work |
I've created a mock up for importing and exporting transactions here I was thinking of not making it possible to export non saved transactions - as transactions will have to have a copy saved to be exported to another user anyway. |
I had a feedback session with @EdMoffatt and Shaun yesterday and they gave some helpful feedback. Lots of words incoming! Summary
The main thing that came up to clarify would be the behaviour of saving transactions https://xd.adobe.com/view/8cb597b2-97c5-474b-4463-c1971452eb80-fa84/ Recap on saving transactions
Scenario 1 - Submitting changes to saved transaction without saving the changes
Scenario 2 - User changes info and saves without submitting
Scenario 3 - User makes changes and doesn't save/submit
@cazfletch Is the ability to add more than one set of both arguments and transient data useful? If so, would the current layout be better - so adding sets of transient data and arguments individually. Or, would dividing the transient data and arguments up into sets - so they come in pairs like this. |
I don't think having sets of arguments is useful, i think if you would save multiple transactions in that case with the different arguments |
Lots of great feedback from Joe Payne and Tom Appleyard!
I'll start acting on this on Monday. |
An update on how the hi fis are coming along.
Any comments are welcome! |
Closing - this works has been completed. |
Submitting a transaction is ok if you just have strings as parameters but it is not ideal if you have objects especially if they are large objects. We should provide a way of submitting a transaction more easily with objects. It would also be good to be able to repeatedly submit transactions without having to go through the process of typing in the parameters again.
The text was updated successfully, but these errors were encountered: