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.

Sign up for GitHub

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

Jump to bottom

User page: new following section #1750

Merged
merged 17 commits into from
Jul 24, 2017
Merged

User page: new following section #1750

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
52e195d
Add tab. Add accordion. Add style. Pass test scss-lint.
agileontheweb Jun 27, 2017
7e2e94d
Add following tab to user public page.
taitus Jun 30, 2017
556e259
Remove margin bottom in the table inside accordion. Divided into grou…
agileontheweb Jul 3, 2017
b0c571e
Do not display send notification button when proposal is from another…
taitus Jul 13, 2017
cef67c5
Increased test coverage to Following user public page.
taitus Jul 13, 2017
36f9836
Use inflector methods instead of manual replacements.
Senen Jul 18, 2017
249641f
Simpler code to get followable object icon.
Senen Jul 18, 2017
16987e7
Easier conditional usage.
Senen Jul 18, 2017
57c8887
Add method to follows helper to get followable partial name and argum…
Senen Jul 18, 2017
b7ef755
minor refactoring for follows
voodoorai2000 Jul 19, 2017
3be893b
Merge pull request #12 from consul/follows-refactoring
Senen Jul 19, 2017
cba497b
Create new followables helper to have some methods previously on foll…
Senen Jul 19, 2017
c9a6630
Remove popup on follow buttons and remove uneeded code.
Senen Jul 19, 2017
119a6b3
Remove where clause from followable button partial and rename followa…
Senen Jul 19, 2017
3b13350
Remove uneeded initialization after ajax refresh.
Senen Jul 19, 2017
dec7d2d
Add followable specific notices to follows controller actions and ren…
Senen Jul 19, 2017
b7bce2e
Remove complex methods from follow model. Create instance method foll…
Senen Jul 19, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
3 changes: 1 addition & 2 deletions app/controllers/users_controller.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -68,8 +68,7 @@ def load_budget_investments
end

def load_follows
@followable_types = @user.follows.pluck(:followable_type).uniq
@follows = @user.follows
@follows = @user.follows.group_by(&:followable_type)
end

def valid_access?
Expand Down
12 changes: 10 additions & 2 deletions app/helpers/follows_helper.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -62,8 +62,16 @@ def entity_icon(entity)
}.invert[entity]
end

def entity_partial(class_name)
class_name.parameterize.gsub('-','_')
def entity_class_name(followable)
followable.class.to_s.parameterize.gsub('-','_')
end

def render_follow(follow)
followable = follow.followable
partial = entity_class_name(followable)
locals = {entity_class_name(followable).to_sym => followable}

render partial, locals
end

private
Expand Down
7 changes: 3 additions & 4 deletions app/views/users/_following.html.erb
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<ul class="accordion" data-accordion data-allow-all-closed="true">

<% @followable_types.each do |followable_type| %>
<% @follows.each do |followable_type, follows| %>

<li class="accordion-item" data-accordion-item>

Expand All @@ -16,9 +16,8 @@
<div class="accordion-content" data-tab-content>
<table>
<tbody>
<% @follows.where(followable_type: followable_type).each do |follow| %>
<%= render entity_partial(followable_type),
entity_partial(followable_type).to_sym => follow.followable %>
<% follows.each do |follow| %>
<%= render_follow(follow) %>
Copy link
Member Author

Choose a reason for hiding this comment

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

@voodoorai2000 nice refactor! 😍

<% end %>
</tbody>
</table>
Expand Down