j-delaney / planner-helper Goto Github PK

I'm pretty sure error messages don't display properly right now. We should fix this.

In the Planner Helper box there is text that says to report bugs to me. I have been getting bug reports for WebReg through it. It should be made more clear that the bug reports are only for Planner Helper

At the moment, Planner Helper is using AS Grade Distribution which is out of date. We need to upgrade to using CAPE's grade distribution system. An example of that can be found here. These are up-to-date.

I was thinking about a more permanent solution to this naming problem than this. I've noticed that there's more than just these professors that are having naming issues.

We could use a separate file so that gradeDist.js, rmp.js, and cape.js could all use the function.

And instead of using a for loop and checking all of them, we could use a hashmap

Thoughts?

Right now, the planner helper box keeps changing height as more and more data loads. This can be distracting for the user and doesn't look very nice. It would be awesome if we could fix this.

We should set up a testing framework.

Some users are confused when they click "view data" and nothing happens. Instead, it should scroll up so that the Planner Helper box is visible

Right now I manually zip up the appropriate files and upload that .zip to the chrome webstore to release a new version. We need a way to automate that process (including minifying the JS files).

I am not sure if this bug occurs for other professors, but for Professor John Hoon Lee of Hum 3, the extension states that his data could not be found even though the data is the website: http://www.ratemyprofessors.com/ShowRatings.jsp?tid=1106609

When running the extension, the user will run into this issue:

Diving further into this. I've noticed that when I reload the extension from source code, only then does the extension successfully load the data.

I've also tried diagnosing the problem by looking at older versions that I knew were working before, such as commits before mine but I run into the exact same problem

I am suspicious that this has part to do with the issue:

I noticed "Avg Grade Expected" and "Avg Grade Received" sections from CAPE are left out from the plugin display.
Were they intentionally left out?

I've done some work on this but it was rejected from the FireFox store because there is an event listener for DOMNodeInserted. This needs to be removed

Known teacher problems so far:

• Geoff Voelker - CSE120
• Minnes Kemp - CSE21
• John Hoon Lee - HUM3
• Altintas De Callaf, Ilkay - CSE15L

The README should include a section detailing how to get started developing for the project. It should include a project structure and the basic of chrome extensions / how to test them.

Right now the icon is a free icon I found online. If anyone has graphic design skills I'd love to see what you come up with for a new icon!

We need to add a feature to allow the user to compare teachers.