Skip to content

week04-assignment#51

Closed
edetn wants to merge 3 commits into
cewing:masterfrom
edetn:master
Closed

week04-assignment#51
edetn wants to merge 3 commits into
cewing:masterfrom
edetn:master

Conversation

@edetn

@edetn edetn commented Feb 5, 2013

Copy link
Copy Markdown

README

  1. Select the following link - runs book1_cgi.py
    http://block647062-yym.blueboxgrid.com/cgi-bin/book1_cgi.py
  2. To display the database book index - runs book2_cgi.py
    Click the hyper link INDEX OF THE BOOK DATABASE
  3. This will display an Index of books with their title
    Click on the title and it will display the page
  4. Source Code is all contained in assignments/week04/athome/src
    the following files comprise the code
    bookdb.py
    book1_cgi.py
    book2_cgi.py
    book2_wsgi_id1.py
    book2_wsgi_id2.py
    book2_wsgi_id3.py
    book2_wsgi_id4.py
    book2_wsgi_id5.py

edetn added 3 commits January 13, 2013 14:40
@edetn
Updated the echo_server file
9a958a5
@edetn
updated echo_client
5a420cf
@edetn
week04-readme
3fceb6b 
README

1. Select the following link - runs book1_cgi.py
http://block647062-yym.blueboxgrid.com/cgi-bin/book1_cgi.py

2. To display the database book index - runs book2_cgi.py
Click the hyper link INDEX OF THE BOOK DATABASE

3. This will display an Index of books with their title
Click on the title and it will display the page

4. Source Code is all contained in assignments/week04/athome/src
the following files comprise the code
bookdb.py
book1_cgi.py
book2_cgi.py
book2_wsgi_id1.py 
book2_wsgi_id2.py 
book2_wsgi_id3.py 
book2_wsgi_id4.py 
book2_wsgi_id5.py
@edetn

edetn commented Feb 5, 2013

Copy link
Copy Markdown
Author

week-04

@DanRutz

DanRutz commented Feb 18, 2013

Copy link
Copy Markdown
Collaborator

Edet,

 I picked up the pull requests last Monday and responded to most everyone about week 5 and Flaskr so far. On the list of open pull requests, your pull request #51 is the one that shows up. I see this pull request has been open for 13 days. Sorry that it has been open so long.

 Your link below works just fine through the first half of step 3. I wasn’t able to bring up the information on the individual books.

 Because I have been checking Flaskr, I tried the root of your VM and came to the main Flaskr page. But when I click the log_in link I got an Internal Server Error.

 When I cloned your repository this morning, your assignments folder only has two things in it: a week01 folder and a week04-readme.txt with a copy of your pull note below. So I wasn’t able to check your code directly.

 However, from what you wrote in step 4 below, I can see your approach was to make a .py file for each book. I think the idea was to make one file that can work for any of the books.

 I’ll go ahead and close your pull request so that you can make a new one for the week 6 Django assignment. And check with Cris or me in class or anyone else about getting your repository figured out so that you’re not deleting your previous weeks’ work. Thanks,

—Dan

From: edetn [mailto:notifications@github.com]
Sent: Tuesday, February 05, 2013 10:29 AM
To: cewing/training.python_web
Subject: [training.python_web] week04-assignment (#51)

README

  1.  Select the following link - runs book1_cgi.py

    http://block647062-yym.blueboxgrid.com/cgi-bin/book1_cgi.py

  2.  To display the database book index - runs book2_cgi.py

    Click the hyper link INDEX OF THE BOOK DATABASE

  3.  This will display an Index of books with their title

    Click on the title and it will display the page

  4.  Source Code is all contained in assignments/week04/athome/src

    the following files comprise the code
    bookdb.py
    book1_cgi.py
    book2_cgi.py
    book2_wsgi_id1.py
    book2_wsgi_id2.py
    book2_wsgi_id3.py
    book2_wsgi_id4.py
    book2_wsgi_id5.py

You can merge this Pull Request by running

git pull https://github.com/edetn/training.python_web master

Or view, comment on, or merge it at:

#51

Commit Summary

  • Updated the echo_server file
  • updated echo_client
  • week04-readme

File Changes

Patch Links:

@DanRutz DanRutz closed this Feb 18, 2013
@DanRutz DanRutz mentioned this pull request Feb 18, 2013
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@edetn @DanRutz