metakgp / gyft

Get Your Freaking Timetable
https://gyft.metakgp.org
MIT License
56 stars 60 forks source link

Ask the user about generating ics file #91

Closed proffapt closed 1 year ago

proffapt commented 1 year ago

Instead of just printing to use generate_ics.py in the end of gyft.py, prompt the user for confirmation whether they want ics file or not and if yes we can just directly execute the generate_ics.py script.

Same goes for add_events.py

And integrate del_events.py via a cli argument what they want to do. Basically one should only use gyft.py for whatever their need be

Note I am not clear about the role of del_events.py so let's first of al complete the first two tasks and if you know what the other file does please tell me.

chirag-ghosh commented 1 year ago

One issue needs to be taken care of. Suppose there occurs some error in finding the course name. The data.txt wont have it and the user needs to enter it manually. In such a senario, we should warn the user about running generate_ics.py

proffapt commented 1 year ago

One issue needs to be taken care of. Suppose there occurs some error in finding the course name. The data.txt wont have it and the user needs to enter it manually. In such a senario, we should warn the user about running generate_ics.py

These are edge cases, obviously needed to be handled.

proffapt commented 1 year ago

@ashwinpra reply here if you will do it, so that it will be assigned to you

ashwinpra commented 1 year ago

@ashwinpra reply here if you will do it, so that it will be assigned to you

Sure, I can work on it.

ashwinpra commented 1 year ago

I can think of two options:

Second option would be better I suppose

proffapt commented 1 year ago

The second method is preferable.

proffapt commented 1 year ago

Being worked on #96