Skip to content

Fix issue #22 course visibility and replacement drop#23

Closed
ceilf6 wants to merge 1 commit into
developfrom
fix/issue-22-course-loading-replace-drop
Closed

Fix issue #22 course visibility and replacement drop#23
ceilf6 wants to merge 1 commit into
developfrom
fix/issue-22-course-loading-replace-drop

Conversation

@ceilf6

@ceilf6 ceilf6 commented Jul 7, 2026

Copy link
Copy Markdown
Owner

Summary

  • document that target courses must be loaded into the current page DOM and recommend precise course-code search
  • add usage notes for "点此查看更多", course detail expansion, refresh behavior, and replacement-course risk
  • fix replacement-course drop lookup by reading the selected-course right list before falling back to the old teaching-class lookup
  • add a Node regression test for locating the drop button in the selected-course right list

Verification

  • node --check courseGrabber.js
  • node --test
  • git diff --check HEAD~1 HEAD

Fixes #22

@ceilf6

ceilf6 commented Jul 7, 2026

Copy link
Copy Markdown
Owner Author

Closing this develop-targeted PR because repo-guard only runs for PRs targeting master. Recreated the same issue #22 fix as #24 against master.

@ceilf6 ceilf6 closed this Jul 7, 2026
@ceilf6 ceilf6 deleted the fix/issue-22-course-loading-replace-drop branch July 7, 2026 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant