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

Access loan list on user profile page #16

Merged
merged 5 commits into from
Nov 7, 2013

Conversation

adamico
Copy link
Contributor

@adamico adamico commented Nov 7, 2013

Summary of changes:

  1. Added a link to current user profile page in navigation section of application layout
  2. Created a relationship between note and the user who created it
  3. Added notes list in user profile page
  4. Added specs to test access to loan from user profile page
  5. Added specs to test note belonging to user after creation

@akshatpradhan
Copy link
Owner

Change Description - 😄

What does this change do?

This feature creates a profile page for people and on the profile page is a link to the loan.

Profile page
screen shot 2013-11-07 at 11 51 47 am

What testing has been done?

This has been QA'd locally and appropriate tests have been run.

Risk Assessment

What are the risks? Is there a business impact?

This has no impact. It adds a relation between a Note and User.

akshatpradhan added a commit that referenced this pull request Nov 7, 2013
user profile page with link to loan
@akshatpradhan akshatpradhan merged commit c82678c into akshatpradhan:master Nov 7, 2013
@adamico adamico deleted the loan_list_on_profile branch November 13, 2013 13:44
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