Skip to content
This repository has been archived by the owner on Oct 18, 2018. It is now read-only.

Tag and filter meet-with-a-mentor requests (Issue #15) #73

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

Tag and filter meet-with-a-mentor requests (Issue #15) #73

wants to merge 9 commits into from

Conversation

steven-ferguson
Copy link
Contributor

#15

@peterkeen
Copy link

Buncha comments in the last commit.

@steven-ferguson
Copy link
Contributor Author

Updated this pull request with the changes recommended.

@peterkeen
Copy link

Looks great. Sorry I took so long to get back to you.

<h3>Concepts</h3>
<p>Click on a concept to see related requests</p>
<ul class='inline'>
<% Concept.all.each do |concept| %>
Copy link

Choose a reason for hiding this comment

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

The Concept.all call should be in the controller: @concepts = Concept.all.

- Add labels
- Move Concept.all call to controller
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants