Merge pull request #1795 from avcwisesa/topics_view

View For hacktoberfest labeled repo

Author: schneems

app/controllers/pages_controller.rb

@@ -51,6 +51,43 @@ def index
     end
   end
 
+  def topic
+    topic_name = params['topic']
+
+    if not ['hacktoberfest'].include?(topic_name)
+      redirect_back fallback_location: root_path
+      return
+    end
+
+    set_title("Open Source Project Topic: #{topic_name}")
+    set_description(description)
+
+    label = Label.find_by(name: topic_name)
+
+    @repos = Repo.with_some_issues
+                 .includes(:repo_labels)
+                 .where(repo_labels: { label_id: label.id })
+                 .select(:id, :updated_at, :issues_count, :language, :full_name, :name, :description)
+
+    topic_repo_ids = @repos.map { |repo| repo.id }
+
+    @repos = @repos.without_user_subscriptions(current_user.id) if user_signed_in?
+    @repos = @repos.order_by_issue_count.page(valid_params[:page]).per_page(valid_params[:per_page] || 50)
+
+    if user_signed_in?
+      @repos_subs = current_user.repo_subscriptions.page(valid_params[:page]).per_page(valid_params[:per_page] || 50).includes(:repo)
+      @repos_subs = @repos_subs.where(repo_id: topic_repo_ids)
+    end
+
+    respond_to do |format|
+      format.html {}
+      format.json do
+        htmlForPage = render_to_string(partial: "repos_with_pagination", locals: { repos: @repos }, formats: ['html'])
+        render json: { html: htmlForPage }.to_json
+      end
+    end
+  end
+
   private
 
   def description

app/views/application/_nav.html.slim

@@ -12,15 +12,40 @@ nav.application-navigation
       li.application-navigation-list-item= image_tag current_user.avatar_url, class: 'user-profile-image', alt: 'user avatar'
       li.application-navigation-list-item= link_to current_user.github, current_user
       li.application-navigation-list-item= link_to 'University', university_index_path
+      li.application-navigation-list-item
+        div.dropdown
+          = content_tag 'a', 'Topics'
+          div.dropdown-content
+            =link_to 'Hacktoberfest', '/topics/hacktoberfest'
       li.application-navigation-list-item= link_to "Submit Repo", new_repo_path
       li.application-navigation-list-item= link_to "Sign Out", destroy_user_session_path, method: :delete
     - else
       li.application-navigation-list-item= link_to 'About', what_path
       li.application-navigation-list-item= link_to 'University', university_index_path
+      li.application-navigation-list-item
+        div.dropdown
+          = content_tag 'a', 'Topics'
+          div.dropdown-content
+            =link_to 'Hacktoberfest', '/topics/hacktoberfest'
       li.application-navigation-list-item= link_to 'Log in', user_github_omniauth_authorize_path, method: :post
       li.application-navigation-list-item= link_to 'Sign Up', user_github_omniauth_authorize_path, method: :post
 
 div.application-logo
   = link_to root_path
     span.application-logo-image
     span.hide-visually Code Triage Home
+
+
+scss class="class":
+
+  .dropdown:hover .dropdown-content {
+    display: block;
+  }
+  .dropdown-content {
+    display: none;
+    position: absolute;
+    background-color: #f9f9f9;
+    min-width: 160px;
+    box-shadow: 0px 8px 16px 0px rgba(0,0,0,0.2);
+    z-index: 1;
+  }

app/views/pages/topic.html.slim

@@ -0,0 +1,22 @@
+.hero
+  h1.hero-title-primary Topic: #{params["topic"]}
+
+  - unless user_signed_in?
+      = link_to user_github_omniauth_authorize_path, class: 'button', method: :post
+        | Sign up with GitHub
+
+hr.section-break
+
+- if @repos_subs.present?
+  .repos-list-wrapper
+    h2.repos-list-title Repos you are currently helping
+    .hero-repo-list-container
+      = render 'repo_subscriptions/list', repos_subs: @repos_subs
+
+  hr.section-break
+
+.repos-list-wrapper
+  h2.repos-list-title Open source projects on GitHub that need your help.
+
+  section.repo-list-with-pagination
+    = render "repos_with_pagination", repos: @repos

config/routes.rb

@@ -28,6 +28,8 @@
 
   root to: "pages#index"
 
+  get "/topics/:topic", to: "pages#topic"
+
   get 'what'    => "pages#what"
   get 'privacy' => "pages#privacy"
   get 'support' => 'pages#support'

test/functional/pages_controller_test.rb

@@ -26,4 +26,9 @@ class PagesControllerTest < ActionController::TestCase
     assert_equal("no-cache", response.headers["Pragma"])
     assert_equal("Fri, 01 Jan 1990 00:00:00 GMT", response.headers["Expires"])
   end
+
+  test "redirect invalid topic" do
+    get :topic, params: { topic: 'test' }
+    assert_redirected_to root_url
+  end
 end