search

voidlily / timeboard

Georgia Tech CoC TA timesheets
4 stars 1 forks source link

Admin section not RESTful #16

Closed voidlily closed 13 years ago

voidlily commented 13 years ago

See comments in b6e36a003451d04ee507

voidlily commented 13 years ago

Actually it could be okay it might just need better named routes. Possibly admin/students/edit or admin/courses/new or stuff like that.

Tclewis commented 13 years ago

This iteration is oodles more RESTful. Re-open this if its still not to your liking, but I think it's pretty RESTy now.

voidlily commented 13 years ago

What commit did you do this in?

voidlily commented 13 years ago

One of the sample routes shown might be better for how we're trying to do the admin section

  # Sample resource route within a namespace:
  #   namespace :admin do
  #     # Directs /admin/products/* to Admin::ProductsController
  #     # (app/controllers/admin/products_controller.rb)
  #     resources :products
  #   end
voidlily commented 13 years ago

Closing for now

voidlily commented 13 years ago

Done more or less, destroying users/courses might not be very RESTful but time constraints.