search

samking / code-the-change-projects

A projects database for projects at the intersection of CS and social change.
Apache License 2.0
0 stars 1 forks source link

add basic tagging capability #26

Open aostiles opened 10 years ago

aostiles commented 10 years ago

I still need to write a test for tags as well as create tag searching for an instant-typing drop down when creating projects.

samking commented 10 years ago

Looks good! I added some comments.

-Sam

On Sun, May 4, 2014 at 4:04 PM, aostiles notifications@github.com wrote:

I still need to write a test for tags as well as create tag searching for

an instant-typing drop down when creating projects.

You can merge this Pull Request by running

git pull https://github.com/aostiles/code-the-change-projects master

Or view, comment on, or merge it at:

https://github.com/samking/code-the-change-projects/pull/26 Commit Summary

  • add basic tagging capability

File Changes

  • M handlers.pyhttps://github.com/samking/code-the-change-projects/pull/26/files#diff-0(2)
  • M models/project.pyhttps://github.com/samking/code-the-change-projects/pull/26/files#diff-1(15)
  • A models/tag.pyhttps://github.com/samking/code-the-change-projects/pull/26/files#diff-2(6)
  • M views/display_project.htmlhttps://github.com/samking/code-the-change-projects/pull/26/files#diff-3(9)

Patch Links:

Reply to this email directly or view it on GitHubhttps://github.com/samking/code-the-change-projects/pull/26 .

aostiles commented 10 years ago

Fixed most of the issues. I haven't added tests yet or messed with the display of the tags.