lurory / carona-bot

Bot for sharing rides.
MIT License
2 stars 0 forks source link

Use of /lotou without carona published breaks list of caronas #42

Closed teredeby closed 1 year ago

teredeby commented 1 year ago

Describe the bug Use of /lotou <ida / volta> without having published a carona to the same direction inserts "NaN/NaN - undefined undefined" in the list of caronas

To Reproduce Steps to reproduce the behavior:

  1. use /volta to publish a carona
  2. use /lotou ida by mistake

Expected behavior Some lines of "NaN/NaN - undefined undefined" are inserted in the list of caronas. I think the issue of not having an hour associated will prevent the wrong line to be deleted in the next day.

image image image

teredeby commented 1 year ago

EDIT: after the same person publish a new "ida", the bugged lines disappeared.

image

fabianaferreira commented 1 year ago

Hello! I started working on #38, which, as far as I remember, should fix this problem. As for now, the solution to this is to delete the entry from MongoDB manually. I'll get back to this and start testing it, because this is a known issue.