voidlily / timeboard

Georgia Tech CoC TA timesheets
4 stars 1 forks source link

CSV User import #12

Open apmonroe opened 13 years ago

apmonroe commented 13 years ago

Be able to add a list of users from an imported csv list

EDIT After talking with bill, here is the format of the csv course, prismId, email, name

jking31cs commented 13 years ago

Alrighty, so Bill wants this. We would need to talk with him about the main format of the csv, but I am more concerned about just something working in there. So here's my suggestion.

He goes to user admin and clicks on "Mass Add Students" button. He will then copy/paste the csv into a text area, and click the import. Now backend logic, my thought.

  1. I assume that we would mark all students inactive here, and this would repopulate them.
  2. If the user is in the system (based on account), we would mark them as active and save it
  3. If the user is not in the system, create the user and save

So for the sake of testing (this part will probably change), make the format like this: Each row is a student, columns are (name, email, account, course_name)

We will need to talk with Bill about how mass course additions occur as well as professors.

voidlily commented 13 years ago

Controller stuff done in 0dc74aa. Can someone help me write the view and routing?

voidlily commented 13 years ago

Filters in d1db1f4

apmonroe commented 13 years ago

it looks to me like every time it imports it deactivates all the students first. shouldnt we leave students alone that arent in the list?

jking31cs commented 13 years ago

No, because Leahy is planning on doing this once a semester to update the user base, including repeat of students. So it should deactivate all, then reactivate those who are not active

On Sat, Jul 30, 2011 at 3:59 PM, apmonroe < reply@reply.github.com>wrote:

it looks to me like every time it imports it deactivates all the students first. shouldnt we leave students alone that arent in the list?

Reply to this email directly or view it on GitHub: https://github.com/rlipman/timeboard/issues/12#issuecomment-1690296

jking31cs commented 13 years ago

Upon staring at this and seeing how file uploading is a pain in rails, I am going to rewrite this and move it over to admin_controller to use a text_area so that routing it will be easier.