sn: Added command to delete last note from user.

[JakubNvk]

To: @mrshu Cc: @pepol @Adman

[mrshu]

@JakubNvk this only deletes the first sticky note that can be found for a given user. Is that what you really wanted to do? I don't think so.

What I thought you'd do would be having two parameters, say nick and id and this command would then delete the nick's note with id id, provided that the user who requested deletion is the original author.

All you'd need to do would be to check if self.notes[nick][id] exists, if event.meta['nick'] is its sender and then just delete it (hopefuly that should work).

[JakubNvk]

@mrshu ping

[mrshu]

@JakubNvk thanks for doing this but I think you overcomplicated things.

Here is how I'd go about doing it:

1. Check whether the user has some notes in self.notes
2. Check whether the note-id can be an index in self.notes[user]
3. Check whether the user who executed this command is the original sender of the note.
4. If all of the above is true, pop the note and update self.notes.

[Adman]

@mrshu @JakubNvk What I would suggest is that after poping the note, we should check whether the notes list for given user is empty. It true, remove that key from self.notes

[JakubNvk]

@Adman I like that idea! Added it.

[mrshu]

@JakubNvk Thanks for the updates! Apart from those two small nitpicks this looks good!

[JakubNvk]

@mrshu allright, shall I squash?

[mrshu]

@JakubNvk go for it!

[mrshu]

@JakubNvk feel free to merge it too =)