u-guru / uguru

College "Uber for Tutoring" app at texas universities
https://u.guru
0 stars 0 forks source link

Course + Department Addition #4044

Open sam1rm opened 8 years ago

sam1rm commented 8 years ago

Nick

sam1rm commented 8 years ago

Sample

https://www.uguru.me/api/v1/universities/1632/popular_courses

Changes that need to be made

//before
getCourses: function(uni_id) {
            return Restangular
                .one('universities', uni_id).customGET('courses');
        },
Add this one
getPopularCourses: function(uni_id) {
            return Restangular
                .one('universities', uni_id).customGET('popular_courses');
        },

Get this working for one university first

I tested using Tufts -- the data should be pretty accurate and the data size different is significant

Important fields for popular courses to make it even faster

sam1rm commented 8 years ago

Two methods to resolve

Two ways to resolve case where they do not have results

Method one

  1. Show 'one list item Add a Course'
  2. Confirm js alert that says "Add _____ ?"
  3. Empty the input
  4. Show rest of the popular courses w/ the most recent item selected at the top

Method Two

  1. Resolve the GUI thread problem (will require reading) -- i can get to it around 9