Skip to content

Conversation

usernolan
Copy link
Contributor

I'm baaaaaaack!

@@ -0,0 +1,138 @@
# An example script showing the functionality of the TinCan Python Library
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'TinCanPython' or 'Tin Can Python' but not 'TinCan Python' :-).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really?! that makes my drawing kind of awkward.

@brianjmiller brianjmiller self-assigned this Jul 9, 2014
@brianjmiller
Copy link
Member

@nolan330 see updated comments, would like to get this guy fixed up and merged in.

@usernolan
Copy link
Contributor Author

@brianjmiller bri-fi, these changes are made, but will hold off committing until after #74 is merged and ensuring that the import method described by Chaim will translate into the example script. Will rebase with canonical master before requesting punishment

@brianjmiller
Copy link
Member

@nolan330 should be good to go, this is the one remaining.

@usernolan
Copy link
Contributor Author

@brianjmiller Alright, changes made and tested. Take a look--let me know if you want me to squash these commits (was afraid to do it out of order i.e. reverse order again, so i didn't bother)

endpoint= "https://cloud.scorm.com/tc/7A3FJJ1UVT/sandbox/"
version = "1.0.1"
username= "ZJCukGAUeh3FMzTXNHo"
password= "3AweKkB7YwbB4R4oVOo"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doh! Probably should deactivate these and remove them from the repo.

brianjmiller added a commit that referenced this pull request Jul 28, 2014
Added example script and dedicated example resource config file
@brianjmiller brianjmiller merged commit 467c445 into RusticiSoftware:master Jul 28, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants