Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Skills and SkillHoldings MVC #18

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

Skills and SkillHoldings MVC #18

wants to merge 23 commits into from

Conversation

bearpaw7
Copy link

Model, View, and Controller for Skills and SkillHoldings:

Skill contains just the :name string and has many Users through SkillHolding.

User has many Skills through SkillHolding.

SkillHolding contains a (Skill) :level integer and belongs to User and Skill.
This allows a User to have a Skill and will save a Skill :level.

Github issue #4

@brundage
Copy link
Contributor

Would you also add a link from app/views/dashboard.html.erb to the new skills interface?

@bearpaw7
Copy link
Author

Dean, thanks for all your help. I will get to work on these improvements.

@brundage
Copy link
Contributor

Cool. All you have to do is make new commits to your branch and they'll show up here.

beef added 10 commits June 27, 2012 06:36
…hboard page.

Display current path (for debuging) and link to dashboard on all pages at the bottom. Must figure out footers.
New method to show Skill level as a string.
…ch adds a disabling style upon render. Firefox and Safari render properly.

Initial favicon.ico and Gimp XCF
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.

2 participants