Freeseer / freeseer

Designed for capturing presentations at conferences. Pre-fill a list of talks to record, record them, and upload them to YouTube with our YouTube Uploader.
http://freeseer.readthedocs.org
GNU General Public License v3.0
215 stars 110 forks source link

Add list argument to CLI talk command #557

Closed jameskunjoolee closed 10 years ago

jameskunjoolee commented 10 years ago

Project proposal

jameskunjoolee commented 10 years ago

Currently lists title of talks without any sort of formatting. Use: python -m freeseer talk list. Should any other field be printed (such as presenter, event, etc.)? Also, should the print statement be formatted a certain way?

I am also having trouble adding talks via the command python -m freeseer talk.

dideler commented 10 years ago

@jameskunjoolee some of your questions were answered on the mailing list, see https://groups.google.com/forum/#!topic/freeseer/Mof0iLrLgA0.

I am also having trouble adding talks via the command python -m freeseer talk.

What kind of trouble? The command freeseer talk opens the talk editor.

jameskunjoolee commented 10 years ago

Ah, thanks @dideler. For adding talks, I open the talk editor and try adding a talk, but nothing actually gets added to the existing list. It works fine on some of my other branches.

dideler commented 10 years ago

@jameskunjoolee sounds like a bug. Can you open a new issue with steps to reproduce.

zxiiro commented 10 years ago

@dideler it's not a bug. @jameskunjoolee you need to delete your database (presentations.db) or the easy way just reset your freeseer. Either delete ~/.freeseer directory yourself or run "python -m freeseer config reset all".

The reason is the database schema changed after we merged Laura's code so your system is missing the updated schema.

dideler commented 10 years ago

@jameskunjoolee btw, you can add talks via the CLI using the talk add command.

freeseer talk add --title "Title" --speaker "Name" --room "Room" --event "Event"
jameskunjoolee commented 10 years ago

@zxiiro Thanks, it works fine now. @dideler I tried that as well, but now it works after resetting freeseer.

jameskunjoolee commented 10 years ago

I opted to use tabular as @dideler suggested. Should I include this in dev_requirements.txt?

Also, I noticed that since I grab the ID value from the database it doesn't behave exactly like the talk editor's "id" field when removing talks: when you remove a talk in the talk editor the number associated with each talk gets updated, but the printed version will not do this since the ID is directly received from the database.

dideler commented 10 years ago

@jameskunjoolee that should go in requirements.txt, since it's a necessary dependency for all users, not just developers.

dideler commented 10 years ago

Thanks for your contribution @jameskunjoolee.