tracker.get_last_event_for ignores restart events

[nbeuchat]

Rasa version: 1.9.5

Rasa SDK version: 1.9.0 (same issue with master)

Python version: 3.7

Operating system (windows, osx, ...): Unix

Issue:
When calling tracker.get_last_event_for("user", skip=n), events that are before a restart event are included.

This seems to be due to an issue in the applied_events.

if event.get("name") == "restart":
    applied_events = []
elif event.get("name") == "undo":
    undo_till_previous("action", applied_events)
elif event.get("name") == "rewind":
    ...

From the code above, the event name is None. I believe it should get the event as event.get("event") instead.

Command or request that led to error:

# run an action with a `Restarted()` event
# in another action, run:
tracker.get_last_event_for("user", skip=n)

[sara-tagger]

Thanks for raising this issue, @tabergma will get back to you about it soon✨

Please also check out the docs and the forum in case your issue was raised there too 🤗

[tabergma]

@nbeuchat Thanks for raising this issue. You seem to be right. Would you be up for fixing this issue and submitting a PR?

[nbeuchat]

@tabergma sure :-) Should this be based on master?

[tabergma]

Yes, master would be great. Thanks 🚀