In Progress: Delete Results from History about trackobot.com HOT 13 CLOSED

Nice UI. This other issue is related, albeit with slightly different controls: stevschmid/track-o-bot#65

Are we worried about users deleting unfavorable matches? (so their match history looks better than it actually was) Or is that beyond the scope of Track-o-Bot?

from trackobot.com.

I don't think it really matters. The user in only cheating himself or herself if they do that. Currently, there are no aggregate statistics amongst all users, but this could skew the results if such a player-wide statistics feature was added (similar to Hearthstats). I'd expect most people wouldn't be so disillusioned as to fake their results, especially there's no social component to trackobot. More so, I doubt enough people would do it to affect any aggregate statistics.

from trackobot.com.

Is this still being developed? Can I help?
I'm really waiting for this feature ;)

from trackobot.com.

I got pretty busy after starting it so haven't finished yet. But I haven't forgotten. It's nearly done just need to figure out a few bugs.

from trackobot.com.

Take your time!

from trackobot.com.

@stevschmid What do you think of this UI? http://take.ms/NmpfP

I'm wondering if we should make these soft deletes so data is recoverable? Do you have a preference?

I still need to figure out how to handle arena results since the results belong to an arena run. I'm not sure what types of scenarios are existing for trackobot users that have issues with arena results that are causing them to want to delete them; that may help in deciding how to implement the delete feature. Currently, I'd have to update the timestamp behavior in arena list since it only uses the last result date, which would throw an error in the list view after deleting the final result in a run. Also, I would probably have to add the option to delete a whole arena run itself. There also exists the issue if trackobot thinks an Arena run is complete, we'd have to re-open the run if some results are deleted. Do you know if a user has ever reported a single Arena in Hearthstone accidentally being tracked as two runs in trackobot?

Once we square out some of the details for arenas, I can implement that as well. However, we can push the deletion of non-arena events earlier since in both the UI and model I prevent deletion of arena results.

from trackobot.com.

Nice work! Looks pretty cool!

Persistence: No recovery for now.

Arena: The ability to delete an arena run would be cool as well. As for the deletion of results which belong to an arena: Not sure what you mean by the timestamp behavior. Shouldn't it be enough to just delete an associated result? I would keep it really simple because it will be used rarely.

from trackobot.com.

It was kind of a trivial comment in retrospect, but just to clarify:

The Arena index view has: <td class="timestamp"><%= local_time_ago(arena.results.order('created_at DESC').first.created_at) %></td> which would give an error: NoMethodError: undefined method 'created_at' for nil:NilClass

So, it'd just be a simple adjustment to account for Arena runs with no results.

from trackobot.com.

Ah yes. The deletion of the last result of an arena run should also delete the run.

from trackobot.com.

Any updates on this?

from trackobot.com.

Sorry, been lazy. I can send a pull request your way if you want to implement what I have so far, or work off of what I have.

Todo:

• Implement Arena deletions
• Create front-end javascript/integration tests

from trackobot.com.

@gssjr Please create a PR with what you have so far. Maybe somebody else has the time and passion to tackle it.

from trackobot.com.

Upcoming version will have this feature. Thanks @gssjr.

from trackobot.com.