Skip to content

Fix guest devices summary page #9197

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

Merged
merged 1 commit into from
May 29, 2024
Merged

Fix guest devices summary page #9197

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 15 additions & 0 deletions app/controllers/guest_device_controller.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ class GuestDeviceController < ApplicationController
include Mixins::GenericShowMixin
include Mixins::MoreShowActions
include Mixins::GenericSessionMixin
include Mixins::BreadcrumbsMixin

before_action :check_privileges
before_action :get_session_data
Expand All @@ -19,6 +20,12 @@ def model
self.class.model
end

def download_summary_pdf
assert_privileges('embedded_automation_manager_credentials_view')

super
end

def self.table_name
@table_name ||= "guest_device"
end
Expand All @@ -34,4 +41,12 @@ def textual_group_list
]
end
helper_method(:textual_group_list)

def breadcrumbs_options
{
:breadcrumbs => [
{:title => _("Guest Devices"), :url => controller_url},
],
}
end
end
1 change: 1 addition & 0 deletions config/routes.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1308,6 +1308,7 @@
show_list
show
quick_search
download_summary_pdf
],

:post => %w[
Expand Down