Resolve "Potentially unsafe external link" for Sip & Sonder in `_includes/about-page/about-card-sponsors.html`

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#8,825 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
1/5
Thời gian dự kiến
Dưới một giờ
Mức phù hợp với người mới
90/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
html
Lĩnh vực
frontend

Hướng nghiên cứu

Open the file _includes/about-page/about-card-sponsors.html, locate the existing <a> tag for the Sip & Sonder link, and add the rel="noopener noreferrer" attribute to it. Start the project with Docker, navigate to the /about page to confirm the link works as expected and the page layout is unchanged across mobile, tablet, and desktop views. Once confirmed, check off the corresponding item in the parent tracking issue #5129 to complete the task.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Feature: Code Alerts good first issue P-Feature: Project Info and Page role: back end/devOps role: front end size: 0.25pt
Prerequisite
  1. Be a member of Hack for LA. (There are no fees to join.) If you have not joined yet, please follow the steps on our Getting Started page.
  2. Before you claim or start working on an issue, please make sure you have read our How to Contribute to Hack for LA Guide.
Overview

We need to fix a potentially unsafe external link by adding the attribute rel="noopener noreferrer". The problem and solution are similar to those detailed in the CodeQL alerts tracked in #5129; however, the instance addressed by this issue did not result in a CodeQL alert.

Action Items
  • Open the file _includes/about-page/about-card-sponsors.html in your IDE
  • Replace
<a href="https://www.sipandsonder.com" target="_blank" alt="Sip and Sonder">

with

<a href="https://www.sipandsonder.com" target="_blank" alt="Sip and Sonder" rel="noopener noreferrer">
  • Using Docker, check the /about page and confirm that the page remains the same in mobile, tablet, and desktop views as on the current website.
  • Confirm that the Sip and Sonder link still works and opens as expected.
Merge Team
  • When this issue is completed, check off the corresponding item in #5129
Resources/Instructions
  1. GitHub CodeQL documentation
  2. Webpage: https://www.hackforla.org/about/
  3. This issue is part of #5129
Ngôn ngữ chính
JavaScript
Star
365
Fork
872
Merge trung bình
1 ngày 15 giờ
Pull request đã merge (30 ngày)
2

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của hackforla/website

Tất cả issue của hackforla/website

Issue tương tự

Thêm issue về JavaScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.