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ở
@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
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Đánh giá
Issue này chưa được đánh giá.