Merge pull request #2156 from Zelda-Cat/fix-sponsor-link

Fix link to sponsor #2512
codebar · Dec 23, 2024 · 100be32 · 100be32
2 parents 6632e6e + 2fcbdfa
commit 100be32
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/views/admin/sponsors/show.html.haml b/app/views/admin/sponsors/show.html.haml
@@ -16,7 +16,7 @@
       .bg-light.p-3.mb-4
         %dl.row
           %dt= t('.website')
-          %dd= link_to @sponsor.website
+          %dd= link_to @sponsor.website, @sponsor.website, target: '_blank'
 
           %dt= t('.address')
           - if @sponsor.address.present?

diff --git a/spec/features/admin/sponsor_spec.rb b/spec/features/admin/sponsor_spec.rb
@@ -97,7 +97,7 @@
     scenario 'displays all sponsor details' do
       expect(page).to have_content(sponsor.name)
       expect(page).to have_content(sponsor.description)
-      expect(page).to have_link(sponsor.website)
+      expect(page).to have_link(sponsor.website, href: sponsor.website)
       expect(page).to have_content(sponsor.level)
       expect(page).to have_content(ContactPresenter.new(sponsor.contacts.first).full_name)