Skip to content

Commit

Permalink
Merge pull request #1170 from concord-consortium/188024567-remove-rub…
Browse files Browse the repository at this point in the history
…ric-copy-link

fix: Hide rubric export/import/copy links
  • Loading branch information
dougmartin authored Sep 6, 2024
2 parents 2952c71 + dcba53b commit 6a6cad6
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 10 deletions.
6 changes: 3 additions & 3 deletions app/views/import/import.html.haml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
.import_activity_sequence
.header
%span.title Import Activity/Sequence/Glossary/Rubric
%span.title Import Activity/Sequence/Glossary
%span.close.close_link
close (✖)

Expand All @@ -13,8 +13,8 @@
=file_field 'import', 'import', accept: 'application/json', class: 'import_file'
.buttons
-#= link_to "Import", import_activities_path, {:remote => true, :class => 'btn btn-primary'}
=submit_tag "import", class: "import btn btn-primary"
=submit_tag "Import", class: "import btn btn-primary"
%a.btn.btn-primary.close
cancel
Cancel
.message
=@message
9 changes: 5 additions & 4 deletions app/views/rubrics/_index_item.html.haml
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,11 @@
= rubric.name
%div.action_menu_header_right
%ul.menu
- if can? :export, rubric
%li.copy= link_to "Export", export_rubric_path(rubric)
- if can? :duplicate, rubric
%li.copy= link_to "Copy", duplicate_rubric_path(rubric)
-# disable export and copy for now since the underlying S3 file is not exported or copied
-#- if can? :export, rubric
-# %li.copy= link_to "Export", export_rubric_path(rubric)
-# - if can? :duplicate, rubric
-# %li.copy= link_to "Copy", duplicate_rubric_path(rubric)
%li.edit= link_to rubric.can_edit(current_user) ? "Edit" : "View", edit_rubric_path(rubric)
- if can?(:update, rubric) && rubric.can_delete()
%li.delete= link_to 'Delete', rubric_path(rubric), method: :delete, data: { confirm: "Are you sure you want to delete this rubric? It is currently being used in #{pluralize(rubric.lightweight_activities.length, 'activity')}. You will not be able to undo this action." }
Expand Down
7 changes: 4 additions & 3 deletions app/views/rubrics/index.html.haml
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,10 @@
- if can? :create, Rubric
%a{ :href => new_rubric_path }
%button=t("NEW_RUBRIC_BUTTON")
- if can? :import, Rubric
%a{ :href => import_status_path, 'data-remote'=>"true" }
%button=t("IMPORT")
-# import disabled as export doesn't contain the S3 file info to import
-#- if can? :import, Rubric
-# %a{ :href => import_status_path, 'data-remote'=>"true" }
-# %button=t("IMPORT")

#official_listing_heading.pagination_heading
%h1 Rubrics
Expand Down

0 comments on commit 6a6cad6

Please sign in to comment.