search

prog-o-meter / prog-o-meter

The prog-o-meter is for tracking your progress during a #100DaysofCode challenge (or any other #100DaysofX challenge) Made for newbies by newbies. Everyone is more than welcome to contribute. Project is for learning workflow and team cooperation in software development
MIT License
32 stars 17 forks source link

meter restart #98

Open barabulkit opened 6 years ago

barabulkit commented 6 years ago

today i was writing additional sharing buttons, and i was tired by editing the meter file before testing how do the buttons works, so i thinked that probably there must be an ability to restart a meter after completion, and also right now, if i created a meter, and then i press button "i don't have a meter yet", and inputing the same name, i will just overwrite the progress that have been done, i think its better to check is a meter with such a name exist at the button "i don't have a meter", to avoid overwriting progress, or add a warning here, and in the main window, instead of disable day adding button after completion, replace it with restart meter button. If you think that its reasonable changes, i can do it.

lineaba commented 6 years ago

I think that adding a process to check if an existing meter exists is a very good idea. For the restart option, I agree that this is something we should work on, but there are a couple of things we need to consider. Mainly, if people are using the logging function, it would be nice to keep them seperate. So for example when a user resets their meter, they should have a new log_file created, for example with the users name, and then a number appendended reflecting their "round", i.e. username_0, username_1, etc. The problem with this if that a user doesn't use the log at all, it seems silly to keep adding new log-files for them. I am open to different suggestions for how to handle this. Maybe @sdkwok has some ideas, since he implemented the log originally?

lineaba commented 6 years ago

So an update: @etiontdn is working on implementing a database, and that includes checking existing users, I believe we might be able to user the same method for the problem you are raising, so there is no need to start working on this. The reset option is still something that we can keep working on.