Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Edit a course from Your Courses with a pencil button

Đang mở
  • Có gì không ổn?
#291 0 bình luận 0 reaction 1 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@Aiden-Trindade đang làm issue này rồi.

Từ ngày 28/10/2026.

Đánh giá

Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức phù hợp với người mới
43/100
Loại issue
Tính năng
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
react, typescript
Lĩnh vực
frontend

Hướng nghiên cứu

Start with landing.tsx, FacultySelector.tsx, and the fullCourseData shape; trace how courses are selected, saved, and used to generate timetables. Check the existing slot multi-selector work in #287 and the layout work in #290, and review how faculty priorities are loaded. Done when course edits save at the same list position, cancel cleanly, and regenerate visible timetables while preserving filters.

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

Mô tả

Scroll down for design

Is your feature request related to a problem? Please describe.

To change a course's slot or faculty, a student deletes it and adds it again. The eye button's popup (#252) only reorders the faculty already picked.

Describe the solution you'd like

A pencil on each course row replaces the eye button and its popup (FacultyPriorityModal). Pressing it puts the selection box into edit mode for that course:

  • Domain, Subject and Slot are filled in, with all of the course's slots selected. Lab-only courses get their morning or evening lab choice.
  • The course's faculty show as ticked, in their current priority order.
  • A yellow "Editing " tag marks the box.
  • Reset becomes Cancel and Confirm becomes Save changes.

While editing:

  • Domain and Subject are locked. Slot and faculty can change.
  • Pressing another course's pencil cancels this edit and starts the new one.
  • Deleting the course being edited cancels the edit.

Save changes:

  • Replaces the course's slots and faculty with what's in the box, so a removed slot is gone.
  • Keeps the course in the same place in Your Courses.
  • If timetables were showing, regenerates them, keeps the filters and goes back to timetable 1.

Cancel leaves the course as it was and clears the box.

Each course remembers the school, programme, batch and domain it was added under, and editing uses those even if the navbar chip (#286) shows a different school. For courses saved before this change, look the subject up in the current school first, then in all schools.

Additional context

Needs #287 (Slot becomes a multi-selector) and the side-by-side layout (#290). Related: #285 (school popup), #252 (the popup this replaces).

Things in the current code to watch for:

  • Confirming a course that's already in the list merges into it and keeps its old slots (facultySelectorOnConfirm in landing.tsx), so Save changes can't reuse it as is.
  • Saved courses (fullCourseData) don't store a school or domain.
  • FacultySelector already re-ticks saved faculty when the subject and slot match a saved course, but it reads from localStorage. The edit can build on it but should read from the course list.
  • QuickModify.tsx is an unused stub that only logs to the console. Please remove it.

FacultySelector.tsx is also being split up in #266, so expect merge conflicts.

Design
Image
Example

One way to build it (just an idea):

  • Keep an editingCourseId in landing.tsx
  • Pass the course to FacultySelector to fill the form, and swap the button labels while it is set
  • Save changes replaces the course at the same index
<FacultySelector
  editingCourse={selectedCourses.find(c => c.id === editingCourseId)}
  onConfirm={editingCourseId ? saveEdit : facultySelectorOnConfirm}
  onCancelEdit={() => setEditingCourseId(null)}
/>
Ngôn ngữ chính
TypeScript
Star
6
Fork
37
Merge trung bình
11 giờ 13 phút
Pull request đã merge (30 ngày)
55

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 CodeChefVIT/ffcs

Tất cả issue của CodeChefVIT/ffcs

Issue tương tự

Thêm issue về TypeScript

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.