Skip to content
This repository has been archived by the owner on Apr 7, 2020. It is now read-only.

Existing bugs #83

Open
2 of 9 tasks
ryanhugh opened this issue May 1, 2019 · 0 comments
Open
2 of 9 tasks

Existing bugs #83

ryanhugh opened this issue May 1, 2019 · 0 comments
Assignees

Comments

@ryanhugh
Copy link
Owner

ryanhugh commented May 1, 2019

Existing bugs:

  • in the termDump.js, save the host data and term data
    • Right now only subjects, classes and sections are saved
  • In the updater, don’t update old terms where sign up has passed
    • Theres a bunch of people watching classes in old terms - these terms are no longer on the site at all. This is causing issues in updater.js.
  • In backend/.../request.js, make sure the content-length in the header of the response matches the actual content length of the response.
    • Sometimes, when we are sending off a bunch of requests at once, requests can come back "cut-short", or like with some part of the response body missing. Lets retry the request if this happens. This is currently prevented with the requiredInBody but thats kindof meh.
  • Fix Rollbar
  • Make sure license header is on every file (can we add a test for this?)
  • Add <doctype> to the index page. It should be there but some stuff breaks when I add it. Lets fix that stuff :)
  • More comments
    • Add some comments to each file that say what each file does
    • What’s ellucian? Why are all the parsing files named ellucian? Maybe it would make more sense to rename the classes/parsers folder
@ryanhugh ryanhugh self-assigned this May 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant