Skip to content
This repository has been archived by the owner on Dec 18, 2019. It is now read-only.

Added screenshots for python3 getting_started.py test step #16

Open
wants to merge 60 commits into
base: master
Choose a base branch
from

Conversation

mwherman2000
Copy link

I've only tested this against the Windows 10 setup steps (previous section of the README.md).

The updated instructions and screen shots assume the python folder is here: /indy-dev/python. Is this the same on Mac and Linux?

@kdenhartog
Copy link
Owner

kdenhartog commented Jan 21, 2019

I'd like to remove all of the images that aren't currently being used to keep the size of this repository smaller. Did you have a plan to use these?

@mwherman2000
Copy link
Author

It's a bit TBD for the unused images. As part of trying to figure out the getting_started.py script, I'm trying to correlate the major and minor steps/tasks in this script with:

I'll likely end up have to regenerate the unused images ...so let me delete them for now.

@kdenhartog
Copy link
Owner

I've gotten a chance to look through this work. I think this is headed in the right direction. It looks like there's a few things that still have TODOs in there. Once you've gotten to a point where you feel like it's ready or you'd like another review can you ping me in the comments here or on rocketchat?

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