Skip to content
This repository was archived by the owner on Sep 3, 2024. It is now read-only.

Comments

Initial Shepherd implementation#124

Open
IanMayo wants to merge 5 commits intomasterfrom
90-walkthrough-implementation
Open

Initial Shepherd implementation#124
IanMayo wants to merge 5 commits intomasterfrom
90-walkthrough-implementation

Conversation

@IanMayo
Copy link
Collaborator

@IanMayo IanMayo commented Apr 1, 2018

Fixes #90

Morphexe added 2 commits April 1, 2018 13:37
Changed eslint parser to allow fat arrow in class
Added walkthrough Steps
@IanMayo
Copy link
Collaborator Author

IanMayo commented Apr 3, 2018

Good work @Morphexe

Feedback:

  • can you make the very first screen 50% wider? That will let it fit on the screen better.
  • step titled "the earlier steps shows how the timestamp...". We've missed out steps 5-8.
  • step titled "Finding data". We need to display this to the right of the target, since it obscures the drop-down lists.
  • final step "Future plans". Not displayed.

IanMayo and others added 3 commits April 3, 2018 12:34
Added missing steps.
Added width fix to the first screen.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants