IntelliTect / IntelliTect/EssentialCSharp.Web

The hard-coded list of routes in `isContentPage` may become brittle; consider extracting the paths into a constant or config and using `Array.includes` for clarity and easier updates.

Đang mở
#747 0 bình luận 1 reaction 2 người được giao Xem trên GitHub

@BenjaminMichaelis đang làm issue này rồi.

Từ ngày 26/4/2026.

  • #1033 của @copilot-swe-agent — đã đóng, không merge
Ngôn ngữ chính
HTML
Star
8
Fork
8
Merge trung bình
17 giờ 23 phút
Pull request đã merge (30 ngày)
61

Mô tả

[nitpick] The hard-coded list of routes in isContentPage may become brittle; consider extracting the paths into a constant or config and using Array.includes for clarity and easier updates.

        const NON_CONTENT_ROUTES = ['/home', '/guidelines', '/about', '/announcements'];
        const isContentPage = computed(() => {
            let path = window.location.pathname;
            return !NON_CONTENT_ROUTES.includes(path);
        });

Originally posted by @Copilot in https://github.com/IntelliTect/EssentialCSharp.Web/pull/667#discussion_r2198566942

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

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.

Đánh giá

Issue này chưa được đánh giá.

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.