mlool / workday-calendar-extension

MIT License
57 stars 22 forks source link

refactor: Update course fetching to prefer extracting course ID from DOM #167

Closed jaskirat-gill closed 3 weeks ago

jaskirat-gill commented 3 weeks ago

What Issue does this PR resolve? (Link to GitHub Issue, approved features and bugs will be given priority)

Please provide a video demo below, or a screenshot and description of the change.

  1. Adding ability to scrape courseID from DOM
  2. Priority of fetching data changed to CourseId from DOM > CourseId from scheduler > Manual Entry
  3. Fetch times significantly approved by ~2x (~700ms to ~350ms) on average No visual changes

    Tag reviewers for the PR below.